Skip to content
This repository has been archived by the owner on Sep 4, 2020. It is now read-only.

Commit

Permalink
Add cordovaDependencies section to package.json (#1232)
Browse files Browse the repository at this point in the history
This reflects `engines` elements from plugin manifest to package.json to comply w/ [new plugin fetching model](https://github.com/cordova/cordova-discuss/blob/master/proposals/plugin-version-fetching.md) and corrects `engine`'s value according to [package.json format](https://docs.npmjs.com/files/package.json#engines)

Its also desirable to add 'protective' entry for next major plugin version to protect end-users from fetching edge versions of the plugin by incompatible version of cordova. See corresponding [discussion on mailing list](http://apache.markmail.org/thread/p73loqtvbzvfzsv5) for more details and reasons behind this.
  • Loading branch information
Vladimir Kotikov authored and macdonst committed Sep 20, 2016
1 parent 14afb94 commit 6b343e7
Showing 1 changed file with 11 additions and 5 deletions.
16 changes: 11 additions & 5 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -29,12 +29,18 @@
"cordova-wp8",
"cordova-browser"
],
"engines": [
{
"name": "cordova",
"version": ">=3.0.0"
"engines": {
"cordovaDependencies": {
"1.8.2": {
"cordova": ">=3.6.3",
"cordova-android": ">=4.0.0",
"cordova-ios": ">=4.1.0"
},
"2.0.0": {
"cordova": ">100"
}
}
],
},
"author": "Adobe PhoneGap Team",
"license": "APL",
"scripts": {
Expand Down

0 comments on commit 6b343e7

Please sign in to comment.