Skip to content

build(deps): @eslint/* の major も ignore に含める - #356

Open
hal-shu-sato wants to merge 1 commit into
mainfrom
build/ignore-eslint-scope
Open

build(deps): @eslint/* の major も ignore に含める#356
hal-shu-sato wants to merge 1 commit into
mainfrom
build/ignore-eslint-scope

Conversation

@hal-shu-sato

Copy link
Copy Markdown
Member

何を

.github/dependabot.ymlignore@eslint/* を追加します。

なぜ

#352eslint の major を除外しましたが、@eslint/js を入れ忘れていました。両者は同じバージョンで同時にリリースされます。

apm-data で同じ設定を入れた数時間後に、dependabot が実際にこの穴を突いてきました(team-apm/apm-data#1004)。

-    "@eslint/js": "^9.39.5",
+    "@eslint/js": "^10.0.1",

eslint は 9 のまま残るので、同じリリースの片割れだけが major をまたぐ状態になります。ここでも同じことが起きます。

@eslint/* に広げれば対になって動くので、eslint 9 → 10 は単独の PR として意図的に上げられます。

関連

apm-data は team-apm/apm-data#1010、apm-web にも同じ PR を出しています。apm 本体は @eslint/js を依存に持っていないため該当しません。

#352 excluded major updates for eslint but not for @eslint/js, which ships
at the same versions, so a major could land on one half of the pair while
the other stayed behind. team-apm/apm-data saw exactly that within hours
of the equivalent change: team-apm/apm-data#1004 raises @eslint/js to
10.0.1 on its own and leaves eslint at 9.39.5.

Widening the entry to @eslint/* keeps the pair moving together, so the
eslint 9 -> 10 step is taken deliberately as a single change.

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
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.

1 participant