cheminfo / eslint-config

Shared ESLint config for cheminfo and ml.js projects
MIT License
3 stars 4 forks source link

Problem with eslint 8 and jsdoc ? #22

Closed lpatiny closed 2 years ago

lpatiny commented 3 years ago
npm ERR! Could not resolve dependency:
npm ERR! peer eslint@"^6.0.0 || ^7.0.0" from eslint-plugin-jsdoc@36.1.1
npm ERR! node_modules/eslint-plugin-jsdoc
npm ERR!   peer eslint-plugin-jsdoc@"^36.1.1" from eslint-config-cheminfo@7.1.0
npm ERR!   node_modules/eslint-config-cheminfo
npm ERR!     dev eslint-config-cheminfo@"^7.1.0" from the root project

@targos I get this error after updating all the dependencies

targos commented 3 years ago

It's weird that you get an error. I just tried with is-any-array and only got a warning: https://github.com/cheminfo/is-any-array/runs/3908101312

lpatiny commented 3 years ago

I have a similar problem in this project: https://github.com/zakodium/service-reference-products/runs/3908178869

But this is a native module project. Don't know if it is related

targos commented 3 years ago

OK I will revert the change and wait for them to support eslint 8

targos commented 3 years ago

Can you try again with version 7.1.1? hopefully I found a hack to fix it