search

paleite / eslint-plugin-diff

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

Fix renamed and modified files not handled. #21

Closed mathieutu closed 2 years ago

mathieutu commented 2 years ago

Hi,

Thanks for this project!

I've just seen that if you modify and rename a file, git will by default consider it as renamed if less than 50% of it changed.

Here, we want to lint every file whose content has changed, so we have to add the -M100% attribute.

You can see the doc about that here.

Thanks!

mathieutu commented 2 years ago

@paleite pinging just in case you don't watch the repo (it happened to me a lot! 😅)

paleite commented 2 years ago

@mathieutu Wow, this is awesome! Thanks a lot!

I was wondering how one could solve that issue, and I'm really grateful that you've contributed 🙂 Thanks!

paleite commented 2 years ago

Published: https://www.npmjs.com/package/eslint-plugin-diff/v/1.0.13-5

currently still in next, but will be released as stable soon

Again, thanks for your contribution 🙂