You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@cordova.apache.org by kelvindart <gi...@git.apache.org> on 2017/10/18 22:51:27 UTC
[GitHub] cordova-plugin-file pull request #220: CB-13294 Remove cordova-plugin-compat
GitHub user kelvindart opened a pull request:
https://github.com/apache/cordova-plugin-file/pull/220
CB-13294 Remove cordova-plugin-compat
### Platforms affected
cordova-android@6.3.0
### What does this PR do?
Removes the dependency on cordova-plugin-compat, and includes checks for min platform version.
### What testing has been done on this change?
Testing 1:
- Add local plugin to project containing cordova-android@6.3.0
- Verify plugin installs, and app runs.
Testing 2:
- Add local plugin to project containing cordova-android@6.2.3
- Verify plugin does not install.
- Console log as follows:
```
Plugin doesn't support this project's cordova-android version. cordova-android: 6.2.3, failed version requirement: >=6.3.0
Skipping 'cordova-plugin-file' for android
```
This is desirable - this change stops the plugin being installed on `<cordova-android@6.3.0`
### Checklist
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) in the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform affected.
- [ ] Added automated test coverage as appropriate for this change.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/kelvindart/cordova-plugin-file master
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-plugin-file/pull/220.patch
To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:
This closes #220
----
----
---
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org
[GitHub] cordova-plugin-file pull request #220: CB-13294 Remove cordova-plugin-compat
Posted by asfgit <gi...@git.apache.org>.
Github user asfgit closed the pull request at:
https://github.com/apache/cordova-plugin-file/pull/220
---
---------------------------------------------------------------------
To unsubscribe, e-mail: commits-unsubscribe@cordova.apache.org
For additional commands, e-mail: commits-help@cordova.apache.org