Add eslint-plugin-import-order-autofix
Closed this issue · 8 comments
@robertrossmann Any thoughts on this?
There's a reported issue with it that it removes lines (bad)
AlexJuarez/eslint-plugin-import-order-autofix#3
But it seems to be fixed in this fork https://github.com/valentin-nemcev/eslint-plugin-import-order-autofix
I am hesitant to include a plugin with such low usage, inactive maintainer (last commit Jan 17, 2017 😱) and a bug which removes code... Especially if adding it to the project takes about 2 lines of code.
I agree with @robertrossmann here. However there is a rule in eslint-plugin-import, which seems to achieve same functionality - import/order.
Although it's not auto-fixable (which I don't mind anyway, because people will learn a good practice), it enforces consistency.
I will close this for now in favour of import/order. I bet someone will eventually implement autofix for the import/order rule. ❤️
Wanna send PR?
@prichodko PR for what...?
For enabling import/order...?
FYI, import/order is now auto-fixable. 🎉 I'll consider enabling the rule in this ruleset (I like the idea, I think I just did not want to bother people to fix all the imports by hand), however in the meantime feel free to enable it on a per-project basis!