-
Notifications
You must be signed in to change notification settings - Fork 26.7k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ESLint Error During Vercel Deployment #36693
Comments
Just curious, does this also happen if you upgrade |
Hello!
|
Thanks for looking into this! I just re-ran a deployment and can confirm that the ESLint is error is still being displayed in the build output.
Here is the entire build output if that is helpful.
|
Having the same issue. It's working locally, but in my CI pipeline it gives the error |
Is there any workaround this issue? |
I don't think this is isolated to Vercel deployment. When I run This error goes away if I downgrade to relevant package.json entries. {
// .. relevant prod
"next": "latest",
"next-transpile-modules": "^9.0.0",
"react": "^17.0.2",
"react-dom": "^17.0.2",
"react-redux": "^7.2.0",
// ... relevant dev
"@types/node": "^16.9.1",
"@types/react": "^17.0.21",
"@types/react-dom": "^17.0.9",
"@types/react-redux": "^7.1.18",
"eslint": "^8.15.0",
"eslint-config-next": "^12.1.5",
"ts-node": "^10.2.1",
"typescript": "^4.3.4",
} |
@aaronmbos is your deployment running Node 12? If so I think increasing it will work 👍 |
Thank you! My Vercel build and deployments were targeting Node 12. I bumped to 16.x to match my local dev environment and the error is now longer displayed. |
confirmed upgrading to node v16 resolves the error locally. Thanks @hanneslund. |
This closed issue has been automatically locked because it had no new activity for a month. If you are running into a similar issue, please create a new issue with the steps to reproduce. Thank you. |
Verify canary release
Provide environment information
What browser are you using? (if relevant)
Not relevant
How are you deploying your application? (if relevant)
Vercel
Describe the Bug
I recently set up ESLint in my Next.js app via
next lint
. Runningnext lint
works correctly in my local development environment. However, when deploying that application an ESLint error is displayed in the build log output. The exact error is:In my local environment:
When deploying on Vercel:
This error does not stop the deployment, but I would like linting to work properly so that I can be sure the build will broken for linting errors.
Here is my current
.eslintrc.json
Here is my
package.json
(I tried the canary version but didn't have any luck so I reverted to 12.1.6)Expected Behavior
I would expect linting not to produce this error during deployment so that I can be confident that linting is working properly and that the build will be broken should linting errors exist. While the error does not stop the deployment from happening, it could be misleading for the error to be displayed in the build log output.
To Reproduce
I'm not sure if this particular issue is reproducible in any other repositories, but I can confirm it does happen in all of my attempted production and preview deployments. I do have my app on Vercel integrated with GitHub to do production deploys on pushes to
main
and preview deploys when pushing to other remote branches.The steps to reproduce this would be to configure ESLint with specified package versions and deploy via Vercel.
Here is a link to my repo.
The text was updated successfully, but these errors were encountered: