Skip to content

Conversation

@nanae772
Copy link
Contributor

@nanae772 nanae772 commented Mar 6, 2025

内容

表題の通りです。

README.mdとCONTRIBUTING.mdのキャッシュについての案内はコピペになってしまっていますが、どちらも見る人がいそうなので書いておきました。

関連 Issue

close #2592

スクリーンショット・動画など

その他

@nanae772 nanae772 requested a review from a team as a code owner March 6, 2025 10:01
@nanae772 nanae772 requested review from Hiroshiba and removed request for a team March 6, 2025 10:01
@voicevox-preview-pages
Copy link

voicevox-preview-pages bot commented Mar 6, 2025

🚀 プレビュー用ページを作成しました 🚀

更新時点でのコミットハッシュ:ee7ab95

package.json Outdated
"scripts": {
"// --- lint ---": "",
"lint": "eslint .",
"lint": "eslint --cache",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

flat configuration fileがある場合は.はあっても無くても同じだと公式ドキュメントに書いてあったため、lint:inspectorとの統一感を出すために何となくで消してしまいました。
https://eslint.org/docs/latest/use/command-line-interface#run-the-cli

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

PR Overview

This PR updates documentation to inform contributors about ESLint’s cache behavior in the repository, detailing when and how the cache file is created and managed.

  • Added instructions for ESLint cache creation and removal in CONTRIBUTING.md.
  • Added corresponding instructions in README.md.

Reviewed Changes

File Description
CONTRIBUTING.md Added information about the ESLint cache file and its management.
README.md Included similar instructions regarding the ESLint cache behavior.

Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.

Copy link
Member

@sevenc-nanashi sevenc-nanashi left a comment

Choose a reason for hiding this comment

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

特に変なところはなさそう。

@nanae772
Copy link
Contributor Author

nanae772 commented Mar 8, 2025

レビューありがとうございます。
#2581 との兼ね合いでコンフリクトが発生していたようなので解消しました。

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!!

eslintの実行が重いの結構気になってたので助かりました!!

@Hiroshiba Hiroshiba enabled auto-merge March 8, 2025 03:46
@Hiroshiba Hiroshiba added this pull request to the merge queue Mar 8, 2025
Merged via the queue into VOICEVOX:main with commit a3e90e4 Mar 8, 2025
12 checks passed
@nanae772 nanae772 deleted the enable-eslint-cache branch March 8, 2025 04:14
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.

ESLintの--cacheを有効化して2回目以降のlintにかかる時間を短縮する

3 participants