ESLint dep update fails #52521

Activity

1. targos

   added

   dependenciesPRs that add, update, or configure Node.js dependencies.PRs that add, update, or configure Node.js dependencies.

   on Apr 14, 2024
2. wraithgar commented on Apr 14, 2024

   

   wraithgar

   on Apr 14, 2024

   Contributor

   More actions

   I can't reproduce with this script, however this script is doing several either wrong or meaningless things.

   This does nothing:

   uninstall --install-links=false --ignore-scripts eslint-plugin-jsdoc eslint-plugin-markdown @babel/core @babel/eslint-parser @babel/plugin-syntax-import-attributes
   

   This is definitely not recommended.

   cd node_modules/eslint
   

   What exactly are you trying to do in this script?

   As far as the error. I can't reproduce it locally. I get an ERESOLVE which makes sense as the error you see appears to be npm having trouble explaining the ERESOLVE error.

   If we can reproduce the error we can fix it. I'd also suggest revisiting your script to see what you are actually wanting it to do here.

   The peer conflict I get locally is:

   npm ERR! code ERESOLVE
   npm ERR! ERESOLVE unable to resolve dependency tree
   npm ERR! 
   npm ERR! While resolving: eslint@9.0.0
   npm ERR! Found: eslint@9.0.0
   npm ERR! node_modules/eslint
   npm ERR!   dev eslint@"file:." from the root project
   npm ERR! 
   npm ERR! Could not resolve dependency:
   npm ERR! peer eslint@"^7.5.0 || ^8.0.0" from @babel/eslint-parser@7.24.1
   npm ERR! node_modules/@babel/eslint-parser
   npm ERR!   @babel/eslint-parser@"*" from the root project
   
3. wraithgar commented on Apr 14, 2024

   

   wraithgar

   on Apr 14, 2024

   Contributor

   More actions

   If you feel you have a reproducible npm bug please submit an issue.
4. targos commented on Apr 15, 2024

   

   targos

   on Apr 15, 2024

   MemberAuthor

   More actions

   I can reproduce locally with the update-eslint.sh script, but I'm not sure it makes sense to submit an issue if it's not using npm in a supported way.
5. targos

   closed this as completedon May 27, 2024