-
Notifications
You must be signed in to change notification settings - Fork 116
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Renovateの設定を直す #637
Renovateの設定を直す #637
Conversation
7ed1f86
to
8d088a7
Compare
8d088a7
to
bfdbd82
Compare
なんかこれで大丈夫そうなのでdraft外します。 |
"regexManagers": [ | ||
"customManagers": [ | ||
{ | ||
"customType": "regex", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
regexManager
というのは古く、今はcustomManagers
というらしい。
"groupSlug": "others", | ||
"groupName": "Others", | ||
"excludePackagePatterns": "^Rust$", | ||
"dependencyDashboardApproval": true |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
これで明示的にdashboardから起動しない限り、PR以前にpushもされなくなるらしい。
(実際branchも作られなくなった)
これがfalse
(デフォルト値)だと、このようなPRが飛んでくる。
qryxip#8
"fileMatch": [ | ||
"^rust-toolchain$" | ||
], | ||
"matchStrings": [ | ||
"(?<currentValue>\\d+\\.\\d+\\.\\d+)" | ||
], | ||
"depNameTemplate": "Rust", | ||
"lookupNameTemplate": "rust-lang/rust", | ||
"packageNameTemplate": "rust-lang/rust", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lookupNameTemplate
というのはドキュメントにもschemaにもなく、Renovateの実装から拾ってきたものであるが、packageNameTemplate
(ちゃんとドキュメントもある)でその役割ができるらしいので置き換え。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!!
メモ書きもありがとうございます、助かります!!
json5にしてコメント書いてメモ書き足していっとくと将来の僕たちを助けてくれるような気がしました。
ちょっとプルリク作ってみます!
VSCodeがデフォルトでjson5ダメだったので諦めました 😇 今回の設定だとRustはOK、Rust以外はなしという形ですが、一律でなしにしたあとRustだけOKにする形もできるのかなとちょっと思いました。 |
内容
#470 (comment)をなんとかします。
関連 Issue
ref #470
その他
fork(qryxip/voicevox_core)での確認はこれからします。