-
Notifications
You must be signed in to change notification settings - Fork 344
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
refactor: rename createComponent as defineComponent #230
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.
We would need to keep a createComponent
function that shows a message saying the method is now named defineComponent
to allow people upgrading without breaking their existing code
16aed1b
to
666db95
Compare
@posva 👍 I pushed a fixup commit introducing retro-compat and a warning in dev mode. Let me know what you think. |
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.
Looks good to me! Thank you
ping @liximomo
Could you please fix the merge conflicts to reflect the changes added in #189? |
@sodatea I merged master into the branch. I used |
a91c945
to
c236f4f
Compare
Since Vue `3.0.0-alpha.1`, `createComponent` has been renamed `defineComponent`. See vuejs/core@1c4cdd8
c236f4f
to
3ce8533
Compare
@posva I properly rebased to squash my extra fixup commit and solve the conflict, using |
Since Vue `3.0.0-alpha.1`, `createComponent` has been renamed `defineComponent`. See vuejs/core@1c4cdd8
Since Vue
3.0.0-alpha.1
,createComponent
has been renameddefineComponent
.See vuejs/core@1c4cdd8