-
-
Notifications
You must be signed in to change notification settings - Fork 695
Add slots+expose to vue/order-in-components default order
#2153
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
✔️ LGTM!
But since this is a breaking change, maybe we should first release a last v9.x version without this PR merged, so we don't force people to upgrade to a major version to use the new features.
Also, we could think about other breaking changes that we may want to accumulate into one v10 release.
|
Agree with you. I think this PR should be released with v10. |
|
@ota-meshi I changed the target branch to |
9c4fe7d to
225738f
Compare
…and `expose` defaults
225738f to
3abdc78
Compare
vue/order-in-components rule to enforce ordering on slots and expose defaultsslots+expose to vue/order-in-components default order
defineSlotsandslotsoption #2125slotstovue/order-in-componentsdefault order #2399