-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
feat: bump plugin version for next major #749
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.
requested two changes
package.json
Outdated
"cordova-android": "<10.0.0", | ||
"cordova-ios": ">=5.1.0", | ||
"cordova": ">=9.0.0" | ||
}, | ||
"6.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.
6.0.0 should include new engines and the >100 should be for a 7.0.0 entry
plugin.xml
Outdated
@@ -31,7 +31,7 @@ | |||
|
|||
<engines> | |||
<engine name="cordova" version=">=9.0.0"/> | |||
<engine name="cordova-android" version=">=9.0.0" /> | |||
<engine name="cordova-android" version="<10.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.
Should be
<engine name="cordova-android" version="<10.0.0" /> | |
<engine name="cordova-android" version=">=10.0.0" /> |
ok, nevermind, the changes I requested are actually done on #750 it's confusing that the PR show those changes, but it's probably related to the PR having conflicts |
7db3001
to
52daaa2
Compare
Platforms affected
None
Motivation and Context
Preparing for next major release that will be for Cordova-Android's next major.
Description
Bump package and package-lock to 6.0.0-dev
Testing
Checklist
(platform)
if this change only applies to one platform (e.g.(android)
)