Skip to content

chore: eslint-config-prettier を削除 - #28

Merged
nemuvski merged 2 commits into
mainfrom
chore/update-eslint-config-prettier
Sep 3, 2026
Merged

chore: eslint-config-prettier を削除#28
nemuvski merged 2 commits into
mainfrom
chore/update-eslint-config-prettier

Conversation

@cursor

@cursor cursor Bot commented Sep 3, 2026

Copy link
Copy Markdown

何を変更したか

eslint-config-prettier を削除しました。バージョンアップは行いません。

  • package.json / yarn.lock から eslint-config-prettier を除去
  • eslint.config.js から import と .prepend(eslintConfigPrettier) を除去
  • vue/html-self-closing を off にする既存の override はそのまま残しています

理由

  • @nuxt/eslint-config はデフォルトで stylistic / formatting ルールを有効にしません(features.stylistic を明示した場合のみ)
  • このリポジトリは createConfigForNuxt({}) のため、Prettier と衝突する ESLint 整形ルールは元から入っていません
  • そのため eslint-config-prettier は実質不要です
  • なお従来の設定は .prepend() していたため、仮に衝突ルールがあっても Nuxt 側のルールが後勝ちになっていました

.prettierrc 自体は残しています(singleQuote: true)。Prettier 本体はもともと依存に含まれていません。

Migration / 設定変更

ESLint 設定から当該パッケージを外しただけです。追加の migration はありません。

検証

  • yarn install — 成功
  • yarn lint — 成功

cursoragent and others added 2 commits September 3, 2026 00:55
Raise the declared range so compromised 10.1.6/10.1.7 versions are
outside the allowed set. 10.1.8 is the clean republish of 10.1.5.

Co-authored-by: K.Utsunomiya <nemuvski@users.noreply.github.com>
@nuxt/eslint-config does not enable stylistic rules by default, so
eslint-config-prettier is unnecessary. Drop the dependency and its
ESLint config import.

Co-authored-by: K.Utsunomiya <nemuvski@users.noreply.github.com>
@nemuvski nemuvski changed the title chore: eslint-config-prettier を 10.1.8 へ更新 chore: eslint-config-prettier を削除 Sep 3, 2026
@nemuvski
nemuvski marked this pull request as ready for review September 3, 2026 01:08

@nemuvski nemuvski left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

@nemuvski
nemuvski merged commit 59e7a34 into main Sep 3, 2026
1 check passed
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