You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@cordova.apache.org by riknoll <gi...@git.apache.org> on 2016/07/29 23:27:04 UTC
[GitHub] cordova-lib pull request #472: CB-11645: Check for getEditConfigs before inv...
GitHub user riknoll opened a pull request:
https://github.com/apache/cordova-lib/pull/472
CB-11645: Check for getEditConfigs before invoking it on pluginInfo
Adding a guard to prevent a type error when platforms and CLI have mismatching cordova-common versions. The issue here is that the `PluginInfo` object crosses the platform API barrier between the two so when 5fb8dffb2dc92e4d08d286432f23299da0e812 added `getEditConfigs`, it ended up being a breaking change. This just checks for the function and ignores `edit-config` changes if it is not defined. More permanent fix might be to move its functionality elsewhere so that it does not cross the platform API barrier. Plugins that rely on `edit-config` (which should be none at this point) should [specify the platform and cordova cli version they need](http://cordova.apache.org/docs/en/latest/guide/hybrid/plugins/index.html#specifying-cordova-dependencies) in package.json.
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/riknoll/cordova-lib edit-config-fix
Alternatively you can review and apply these changes as the patch at:
https://github.com/apache/cordova-lib/pull/472.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 #472
----
commit 72074609c931177db9810723b4ee3425cf9e6b58
Author: Richard Knoll <ri...@microsoft.com>
Date: 2016-07-29T23:19:11Z
CB-11645: Check for getEditConfigs before invoking it on pluginInfo
----
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-lib issue #472: CB-11645: Check for getEditConfigs before invoking i...
Posted by riknoll <gi...@git.apache.org>.
Github user riknoll commented on the issue:
https://github.com/apache/cordova-lib/pull/472
Closing in favor of #473
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-lib issue #472: CB-11645: Check for getEditConfigs before invoking i...
Posted by codecov-io <gi...@git.apache.org>.
Github user codecov-io commented on the issue:
https://github.com/apache/cordova-lib/pull/472
## [Current coverage](https://codecov.io/gh/apache/cordova-lib/pull/472?src=pr) is 80.66% (diff: 100%)
> Merging [#472](https://codecov.io/gh/apache/cordova-lib/pull/472?src=pr) into [master](https://codecov.io/gh/apache/cordova-lib/branch/master?src=pr) will not change coverage
```diff
@@ master #472 diff @@
==========================================
Files 68 68
Lines 5399 5399
Methods 856 856
Messages 0 0
Branches 1043 1043
==========================================
Hits 4355 4355
Misses 1044 1044
Partials 0 0
```
> Powered by [Codecov](https://codecov.io?src=pr). Last update [64e67ec...c56337d](https://codecov.io/gh/apache/cordova-lib/compare/64e67ecf74e6d730b488f13fa0c638a673623738...c56337d080fd8e97828f5abca8a35cea6f263239?src=pr)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-lib issue #472: CB-11645: Check for getEditConfigs before invoking i...
Posted by riknoll <gi...@git.apache.org>.
Github user riknoll commented on the issue:
https://github.com/apache/cordova-lib/pull/472
@ktop @stevengill @vladimir-kotikov @TimBarham please take a look
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org
[GitHub] cordova-lib pull request #472: CB-11645: Check for getEditConfigs before inv...
Posted by riknoll <gi...@git.apache.org>.
Github user riknoll closed the pull request at:
https://github.com/apache/cordova-lib/pull/472
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastructure@apache.org or file a JIRA ticket
with INFRA.
---
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org