Skip to content
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

chore: すべての依存をRenovateの対象にする #839

Merged
merged 22 commits into from
Sep 29, 2024

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Sep 27, 2024

内容

次の2つの"group"にまとめる形で、あらゆる依存をRenovateの自動PRの対象にします。

  • "major dependencies" (renovate/major-dependencies)
  • "non-major dependencies" (renovate/non-major-dependencies)

0.y.zのy、0.0.zのzの変更は"major dependencies"側に分類します。現在唯一自動PRの対象にしているRustのバージョンについては、"non-major dependencies"側です。

運用としては二つのPRを普段は保持し続け、更新可能ライブラリを把握する目的に留めるということを考えています。そして6週間おきのRustのリリースのタイミングで"non-major"のPRを「収穫」、"major"の方はしかるべき機会が来たら収穫という風にします。

実行例は次のような感じです。

関連 Issue

Resolves #470.

その他

renovate.json5 Outdated
"matchUpdateTypes": [
"major"
],
"dependencyDashboardApproval": false
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

折角JSON5にしたので、末尾カンマ入れてもいいかも?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

どちらでも良さそう! どちらかというと入れた方がいい気がしますね!

Copy link
Member Author

@qryxip qryxip Sep 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

9d20232 (#839)
JSON5としてフォーマットした…のですがこれは…
image
"renovate.json5"を受け付けると言いつつ実はJSONCまでしか受け付けないとか…? (ありえそう)

[追記] 末尾カンマは大丈夫っぽい?
image

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

いやこれよくわからないな。末尾カンマはOKなのかそうじゃないのか…???

Copy link
Member Author

@qryxip qryxip Sep 28, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

検証したところ「arrayの末尾にあるobjectに対する末尾カンマ」(},)だけ、GitHub App版のRenovateにだけ通らないみたいです。
Discord: https://discord.com/channels/879570910208733277/893889888208977960/1289475175066370099

基本JSON5にしつつGitHub App版には通るようにして、コメントで注意書きという形ですかね…

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

注意書きのコメントはこうなりました。
cdb6d50 (#839)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

えーと今 (09-28 19:00くらい)報告用の最小例を作ろうとしたら再現せず、このPRでの末尾カンマを復活させたらGitHub App版も普通に動きました… 何だかわからないけど一過性のものだった?

6b8b92d (#839)
53e35e5 (#839)

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"Error executing gradle wrapper update command"のメッセージと共にgradle-wrapper.jarが消し飛ばされる件が気になるけど、どういう現象なのかまだわかってないし、Gradle自体のアップデートは別にやることで回避可能ではあるので除外設定はやらなくていいかなと思ってます。

renovate.json5 Outdated Show resolved Hide resolved
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ほぼLGTMです!!

どんな感じの運用になるか楽しみです!!

このリポジトリをウォッチしてる人にとって、通知がうるさすぎなければいいかなと思ってます。
どうやら今の運用だとforce pushされまくる形になっていて、かつそのforce pushはどうやら通知が来ないっぽいので、その仕様が続くうちはこの運用で良いのかなと思いました!

@qryxip qryxip marked this pull request as draft September 28, 2024 05:37
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Sep 28, 2024
@qryxip qryxip marked this pull request as ready for review September 28, 2024 07:37
@qryxip qryxip force-pushed the chore-use-renovate-for-all-deps branch from cdb6d50 to 84c641e Compare September 28, 2024 09:44
@qryxip qryxip marked this pull request as draft September 28, 2024 10:16
@qryxip qryxip marked this pull request as ready for review September 28, 2024 10:23
Copy link
Member

@Hiroshiba Hiroshiba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!!!

調整ありがとうございました!!!
どんな感じの運用感になるか、楽しみです!!

@qryxip qryxip merged commit 521cd54 into VOICEVOX:main Sep 29, 2024
30 checks passed
@qryxip
Copy link
Member Author

qryxip commented Sep 29, 2024

あ!来てしまいましたねforce pushによるpush通知…

@Hiroshiba
Copy link
Member

Hiroshiba commented Sep 30, 2024

おっと!! ちょっともうちょっと様子見してみますか・・・!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Renovateを導入する
2 participants