paleite / eslint-plugin-diff

Run ESLint on your changes only
https://npm.im/eslint-plugin-diff
MIT License
181 stars 14 forks source link

ESLint couldn't find the config "plugin:diff/ci" to extend from. #55

Closed AndyOGo closed 1 year ago

AndyOGo commented 1 year ago

First, this is a really neat idea!

Is your feature request related to a problem? Please describe.

Yes, as follows:

  1. npm install --save eslint-plugin-diff

  2. create a shareable ESLint config and extends from "plugin:diff/ci" as outline here

  3. Oops! Something went wrong! :( ESLint: 8.47.0 ESLint couldn't find the config "plugin:diff/ci" to extend from. Please check that the name of the config is correct.

Describe the solution you'd like

EsLint should be able to find plugin:diff/ci

Describe alternatives you've considered None

Additional context

$ npm ls eslint-plugin-diff
└── eslint-plugin-diff@2.0.1

dist/ci.js is missing from the build, see: https://www.runpkg.com/?eslint-plugin-diff@2.0.1

AndyOGo commented 1 year ago

Alright, there are some fixes available as part of the next tag. npm install --save eslint-plugin-diff@next

$ npm ls eslint-plugin-diff
└── eslint-plugin-diff@2.0.2-2

dist/ci.js exists, see: https://www.runpkg.com/?eslint-plugin-diff@2.0.2-2/dist/ci.js

paleite commented 1 year ago

Thanks for highlighting this. 2.0.2 has now been released, so npm install --save eslint-plugin-diff will work now.