[Update] Make vue/attributes-order fixable
#405
Conversation
* [Update] Make `vue/max-attributes-per-line` fixable * [fix] bug and style * [fix] Switch indent calculation method with node and attribute * [fix] don't handle indentation * [add] autofix test max-attributes-per-line.js
* [Update] Make `vue/order-in-components` fixable This Commit makes `vue/order-in-components` fixable. In case of `The "A" property should be above the "B" property` error, autofix will move A before B * [fix] fail test at [email protected] * [fix] If there is a possibility of a side effect, don't autofix * [fix] failed test at node v4 * [update] use Traverser * [fix] failed test [email protected] * [fix] I used `output: null` to specify "not fix"
There was a problem hiding this comment.
Thank you for working on this @ota-meshi :) Could you please add two more test cases with more than 2 errors, and completely messed order of different properties and the same with different order setting passed.
|
@michalsnik |
|
@ota-meshi @michalsnik Just applied this branch to a codebase with more than 400 attributes-order errors and it worked smoothly. |
|
Music to my ears! Thank you @ota-meshi awesome work :) |
|
Thank you so much @ota-meshi, really looking forward to using this 🎉 Does anyone know when this will be released? Will be a huge time saver for them old codebase. |
|
Sorry guys it took so long... But I just released v4.5.0, enjoy! :) |
This PR makes
vue/attributes-orderfixable. #401In case of
Attribute "A" should go before "B"error message, autofix will move A before B