-
Notifications
You must be signed in to change notification settings - Fork 86
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
update/peer-dependencies #48
Conversation
please bump the vue-class-component dependency, if possible it appears to be working just fine but I haven't run any extensive tests
package.json
Outdated
@@ -64,6 +64,6 @@ | |||
"peerDependencies": { | |||
"vue": "^2.5.0", | |||
"vuex": "^3.0.0", | |||
"vue-class-component": "^6.0.0" | |||
"vue-class-component": "^7.0.0" |
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.
Could you include ^6.0.0
because it should be compatible with both? So it will be ^6.0.0 || ^7.0.0
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.
done!
The current version is already ^6.0.0 and it throws yarn warnings if you
have 7 side by side.
…On Thu, 14 Mar 2019 at 23:30 katashin ***@***.***> wrote:
***@***.**** requested changes on this pull request.
------------------------------
In package.json
<#48 (comment)>:
> @@ -64,6 +64,6 @@
"peerDependencies": {
"vue": "^2.5.0",
"vuex": "^3.0.0",
- "vue-class-component": "^6.0.0"
+ "vue-class-component": "^7.0.0"
Could you include ^6.0.0 because it should be compatible with both? So it
will be ^6.0.0 || ^7.0.0
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#48 (review)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/ASb5jM-vyS3DKMlxTW9plXC9nY-aeiuIks5vWwXCgaJpZM4b0vTs>
.
|
@oakromulo What do you mean by that? |
- retain support for `vue-class-component` *v6* alongside *v7* as a peer dependency
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.
- retain support for
vue-class-component
v6 alongside v7, as suggested
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.
Thank you!
please bump the vue-class-component dependency, if possible
it appears to be working just fine but I haven't run any formal tests