Skip to content

feat(eslint): migrate to ESLint v10#821

Open
YusukeHirao wants to merge 4 commits intodevfrom
feat/eslint-v10
Open

feat(eslint): migrate to ESLint v10#821
YusukeHirao wants to merge 4 commits intodevfrom
feat/eslint-v10

Conversation

@YusukeHirao
Copy link
Copy Markdown
Member

Summary

  • ESLint v9.39.2 → v10.0.0、@eslint/js v9.39.2 → v10.0.1 にアップデート
  • eslint-plugin-eslint-comments を非推奨のため @eslint-community/eslint-plugin-eslint-comments に置換
  • @eslint/compatfixupPluginRules で v10 未対応プラグイン(import-x, jsdoc, comments, sort-class-members)をラップ
  • 非推奨ルール @typescript-eslint/no-var-requires を削除
  • ESLint v10 新ルール no-useless-assignment で検出された prefer-individual-transform-properties のコードを修正

Test plan

  • yarn lint:eslint が通ること
  • ESLint関連テスト(sort-class-members, prefer-top-level-await, Disallow DOMContentLoaded)が通ること
  • prefer-individual-transform-properties のユニットテスト(13件)が通ること

- Update eslint to v10.0.0 and @eslint/js to v10.0.1
- Replace eslint-plugin-eslint-comments with
  @eslint-community/eslint-plugin-eslint-comments
- Add @eslint/compat and wrap v10-incompatible plugins
  with fixupPluginRules (import-x, jsdoc, comments,
  sort-class-members)
- Remove deprecated @typescript-eslint/no-var-requires rule
@YusukeHirao YusukeHirao requested a review from arktds February 9, 2026 13:03
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@cursor
Copy link
Copy Markdown

cursor bot commented Mar 2, 2026

You have run out of free Bugbot PR reviews for this billing cycle. This will reset on March 26.

To receive reviews on all of your PRs, visit the Cursor dashboard to activate Pro and start your 14-day free trial.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@arktds
Copy link
Copy Markdown
Collaborator

arktds commented Apr 13, 2026

@YusukeHirao
eslint-plugin が v9 のままでして何か理由はありますか?

"peerDependencies": {
"eslint": ">=9.0.0"
},
"dependencies": {
"@typescript-eslint/utils": "8.53.1"
},
"devDependencies": {
"@typescript-eslint/parser": "8.53.1",
"@typescript-eslint/rule-tester": "8.53.1",
"eslint": "9.39.2",
"vue-eslint-parser": "9.4.3"
}

関連して、@typescript-eslint/utils が v8.53.1 になっていますが、その peerDependencies で eslint v10 が含まれていないようでして、ただ今の yarn.lock では(おそらく他パッケージの依存によって)@typescript-eslint/utils の v8.56.1 が eslint v10 を含んでいるようです。
そのため、peerDependencies と devDependencies の eslint を v10 に変更して、@typescript-eslint/utils、@typescript-eslint/parser、@typescript-eslint/rule-tester を v8.53.1 から少なくとも v8.56.1 に変更したほうが良さそうです!

@arktds
Copy link
Copy Markdown
Collaborator

arktds commented Apr 13, 2026

@YusukeHirao
@eslint-community/eslint-plugin-eslint-comments の v4.6.0

"@eslint-community/eslint-plugin-eslint-comments": "4.6.0",

の peerDependencies で eslint が v9 までしか明記されていないようです。

    "peerDependencies": {
        "eslint": "^6.0.0 || ^7.0.0 || ^8.0.0 || ^9.0.0"
    },

最新(今だと v4.7.1)の @eslint-community/eslint-plugin-eslint-comments だと v10 も明記されています。
@eslint-community/eslint-plugin-eslint-comments を最新版にアップデートしたほうが良さそうです!

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.

2 participants