Skip to content

Conversation

bgub
Copy link
Contributor

@bgub bgub commented Aug 19, 2025

No description provided.

@ijjk ijjk added created-by: Next.js team PRs by the Next.js team. Documentation Related to Next.js' official documentation. labels Aug 19, 2025
@bgub bgub requested review from ztanner and ijjk August 19, 2025 20:53
@bgub bgub enabled auto-merge (squash) August 19, 2025 20:54
@@ -66,7 +66,7 @@ Then, add the following scripts to your `package.json` file:
"dev": "next dev",
"build": "next build",
"start": "next start",
"lint": "next lint"
"lint": "eslint"
Copy link
Contributor

Choose a reason for hiding this comment

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

The eslint command requires file patterns to specify what should be linted. Without arguments, it won't lint any files. The original next lint command handled this automatically, but when switching to direct eslint usage, you should specify a pattern like:

"lint": "eslint ."

or

"lint": "eslint src/"

This ensures ESLint will actually scan the appropriate files in your project.

Suggested change
"lint": "eslint"
"lint": "eslint ."

Spotted by Diamond

Fix in Graphite


Is this helpful? React 👍 or 👎 to let us know.

@bgub bgub merged commit 91da003 into canary Aug 19, 2025
75 checks passed
@bgub bgub deleted the fix/next-lint-docs branch August 19, 2025 20:56
@github-actions github-actions bot added the locked label Sep 3, 2025
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 3, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
created-by: Next.js team PRs by the Next.js team. Documentation Related to Next.js' official documentation. locked
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants