build: 版表記の追随対象からビルドツールの版を外す - #40
Merged
Merged
Conversation
ビルドツール(Gradle / Maven)の版は配布物へ影響しないが、資料が具体的な版へ 言及していたため、checkVersionMentions がラッパー更新のたびに資料の追随を要求し、 Dependabot のラッパー更新 PR が CI で落ちていた。 追随対象を配布物を左右する版(Kotlin 版・自版)に限り、wrapper properties からの 版導出と Gradle / Maven の置換規則を削除する。資料側はサポート下限のみを残し、 具体的な版への言及を落とす。README の互換表の実測版注記も、リリース宣言時に 同じ理由で乖離するため併せて削除する。 Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
背景
Dependabot の Gradle ラッパー更新 PR(#38 / #39)が CI で落ちる。失敗は
build/build-macosの 1 か所のみで、ci-gateはその集約:versionMentionRulesが wrapper properties から Gradle 版を導出し資料中の表記と照合していたため、ラッパー更新のたびに資料の追随コミットが必須になっていた。ビルドツールの版は配布物へ影響しないため、追随を要求すること自体が過剰である。変更
build.gradle.kts:gradleWrapperVersionの導出と Gradle / Maven の置換規則を削除し、追随対象を配布物を左右する版(Kotlin 版・自版)に限る方針をコメントへ残すdocs/実装ノート.md/docs/test/テスト戦略.md: 実測環境の記述から Gradle / Maven の具体的な版を削除するREADME.md: 互換表の実測版注記を削除する。下限表記Gradle 9+ or Maven 3.9+は維持するREADME の該当行は release 群に属し現在は未検査だが、リリース宣言時に同じ理由で乖離するため併せて解消する。
Kotlin 更新 PR が
syncVersionMentionsの追随を要求する点は設計どおりで変更しない(配布版<KotlinVersion>-<自版>に Kotlin 版が乗るため)。検証
ktfmtCheck/checkVersionMentions/checkMavenVersionが greencheckVersionMentionsに加え、ルートのbuildも green(本 PR にラッパーの更新は含めない)CI が中断していたため、ルートビルドが更新後のラッパーで通ること自体が未検証だった。その確認まで行っている。
🤖 Generated with Claude Code