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.

Sign up for GitHub

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

Jump to bottom

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

Merged
merged 22 commits into from
Sep 29, 2024
Merged

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

merged 22 commits into from
Sep 29, 2024
+90 −41

Conversation

qryxip
Copy link
Member

@qryxip qryxip commented Sep 27, 2024 •
edited
Loading

内容

次の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.

その他

qryxip
qryxip commented Sep 27, 2024
View reviewed changes
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 •
edited
Loading

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 •
edited
Loading

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)

renovate.json5
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自体のアップデートは別にやることで回避可能ではあるので除外設定はやらなくていいかなと思ってます。

Hiroshiba
Hiroshiba reviewed Sep 27, 2024
View reviewed changes
renovate.json5 Outdated Show resolved Hide resolved
Hiroshiba
Hiroshiba reviewed Sep 27, 2024
View reviewed changes
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
Revert "JSON5としてフォーマット"
927d714 
This reverts commit 9d20232.
@qryxip
末尾カンマだけ入れてみる
a5ef979
@qryxip
末尾カンマだけ入れる
432e572
@qryxip
Revert "末尾カンマだけ入れる"
322dcd7 
This reverts commit 432e572.
@qryxip
mapの末尾にカンマを入れてみる
392662b
@qryxip
「arrayの末尾のobject」に末尾カンマを入れてみる
cf101d9
@qryxip
Docker版でrenovate.json5を読んでみる
10ba553
@qryxip
fixup! Docker版でrenovate.json5を読んでみる
701d7fd
@qryxip
Revert "「arrayの末尾のobject」に末尾カンマを入れてみる"
0185235 
This reverts commit cf101d9.
@qryxip
keyの""を外したらどうなるか？
7574139
@qryxip
「arrayの末尾のobject」に対する末尾カンマだけ避けてJSON5化
13904f6
@qryxip
Revert "fixup! Docker版でrenovate.json5を読んでみる"
3e47acf 
This reverts commit 701d7fd.
@qryxip
Revert "Docker版でrenovate.json5を読んでみる"
79521af 
This reverts commit 10ba553.
qryxip added a commit to qryxip/voicevox_core that referenced this pull request Sep 28, 2024
@qryxip
末尾カンマについての注意事項
cdb6d50 
VOICEVOX#839 (comment)
@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
Hiroshiba
Hiroshiba approved these changes Sep 28, 2024
View reviewed changes
Copy link
Member

@Hiroshiba Hiroshiba left a comment •
edited
Loading

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通知…

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Hiroshiba Hiroshiba Hiroshiba approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Renovateを導入する
2 participants
@qryxip @Hiroshiba