-
Notifications
You must be signed in to change notification settings - Fork 3
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
chore: upgrade eslint #155
base: main
Are you sure you want to change the base?
Conversation
with the Run npm ci
npm ERR! code ERESOLVE
npm ERR! ERESOLVE could not resolve
npm ERR!
npm ERR! While resolving: [email protected]
npm ERR! Found: [email protected]
npm ERR! node_modules/eslint
npm ERR! dev eslint@"^9.1.0" from the root project
npm ERR! peer eslint@"^6.0.0 || ^7.0.0 || >=8.0.0" from @eslint-community/eslint-utils@[4](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:5).4.0
npm ERR! node_modules/@eslint-community/eslint-utils
npm ERR! @eslint-community/eslint-utils@"^4.2.0" from [email protected]
npm ERR! @eslint-community/eslint-utils@"^4.1.2" from eslint-plugin-es-x@7.[5](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:6).0
npm ERR! node_modules/eslint-plugin-es-x
npm ERR! eslint-plugin-es-x@"^7.5.0" from [email protected]
npm ERR! node_modules/eslint-plugin-n
npm ERR! dev eslint-plugin-n@"^17.2.0" from the root project
npm ERR! 1 more (eslint-plugin-n)
npm ERR! 5 more (eslint-compat-utils, eslint-config-prettier, ...)
npm ERR!
npm ERR! Could not resolve dependency:
npm ERR! peer eslint@"^2 || ^3 || ^4 || ^5 || ^[6](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:7) || ^7.2.0 || ^8" from [email protected]
npm ERR! node_modules/eslint-plugin-import
npm ERR! dev eslint-plugin-import@"^2.29.0" from the root project
npm ERR!
npm ERR! Conflicting peer dependency: [email protected][7](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:8).0
npm ERR! node_modules/eslint
npm ERR! peer eslint@"^2 || ^3 || ^4 || ^5 || ^6 || ^7.2.0 || ^[8](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:9)" from [email protected][9](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:10).1
npm ERR! node_modules/eslint-plugin-import
npm ERR! dev eslint-plugin-import@"^2.29.0" from the root project
npm ERR!
npm ERR! Fix the upstream dependency conflict, or retry
npm ERR! this command with --force, or --legacy-peer-deps
npm ERR! to accept an incorrect (and potentially broken) dependency resolution.
npm ERR!
npm ERR! See /home/runner/.npm/eresolve-report.txt for a full report.
npm ERR! A complete log of this run can be found in:
npm ERR! /home/runner/.npm/_logs/2024-04-29T[17](https://github.com/elastic/elastic-otel-node/actions/runs/8882442044/job/24387150370?pr=155#step:6:18)_00_46_673Z-debug-0.log It's odd this does not happen in |
Perhaps because apm-agent-nodejs repo has this:
|
Is there an issue or PR on eslint-plugin-import to have it update its peerDeps to include eslint v9? |
There is an issue to add support to and theres is a draft PR import-js/eslint-plugin-import#2996 so the peerDependencies will be updated in a newer version |
Wow, that's a big PR change to eslint-plugin-import. |
Updates
eslint
to v9 and switches to the new flat config.