-
-
Notifications
You must be signed in to change notification settings - Fork 666
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add slots
to vue/order-in-components
default order
#2399
Comments
I think we can do it in the next major version. #2153 |
Ah, I only checked open issues, not PRs. I linked the PR to this issue now. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What rule do you want to change?
vue/order-in-components
Does this change cause the rule to produce more or fewer warnings?
More.
How will the change be implemented? (New option, new default behavior, etc.)?
Add
slots
to default order. Currently, the default order is:eslint-plugin-vue/lib/rules/order-in-components.js
Lines 14 to 76 in e936a15
I'd propose to add the
slots
option afteremits
, in theInterface
category. This is probably a breaking change, but we could argue that theslots
option (which was introduced in Vue 3.3) is not in use for long, so intentionally breaking might be okay.@ota-meshi what do you think?
The text was updated successfully, but these errors were encountered: