[
https://issues.apache.org/jira/browse/CB-12242?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16527507#comment-16527507
]
ASF GitHub Bot commented on CB-12242:
-------------------------------------
raphinesse closed pull request #610: CB-12242 : Use yarn instead of npm
URL: https://github.com/apache/cordova-lib/pull/610
This is a PR merged from a forked repository.
As GitHub hides the original diff on merge, it is displayed below for
the sake of provenance:
As this is a foreign pull request (from a fork), the diff is supplied
below (as it won't show otherwise due to GitHub magic):
diff --git a/src/cordova/plugin/add.js b/src/cordova/plugin/add.js
index 65a75812c..f86c0d9e5 100644
--- a/src/cordova/plugin/add.js
+++ b/src/cordova/plugin/add.js
@@ -89,7 +89,8 @@ function add (projectRoot, hooksRunner, opts) {
variables: opts.cli_variables,
is_top_level: true,
save_exact: opts['save-exact'] || false,
- production: opts.production
+ production: opts.production,
+ manager: opts.manager
};
return module.exports.determinePluginTarget(projectRoot,
cfg, target, fetchOptions).then(function (resolvedTarget) {
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
> Use yarn js instead of npm when adding plugins
> ----------------------------------------------
>
> Key: CB-12242
> URL: https://issues.apache.org/jira/browse/CB-12242
> Project: Apache Cordova
> Issue Type: Improvement
> Components: cordova-fetch
> Reporter: Jacques de Villiers
> Priority: Major
>
> Currently it can take quite long to add certain plugins to my project (using
> cordova plugin add), especially if I need to re-add the plugins. My
> suggestion is to update the cordova cli to start using yarn js instead of npm
> directly.
> When I looked at this page, I realised yarn was just a wrapper for npm, and
> much better at caching packages locally.
> https://shift.infinite.red/npm-vs-yarn-cheat-sheet-8755b092e5cc#.jiz27n1hc
> I would imagine this change would be relatively straightforward, and would be
> a massive win for the cli.
> I was thinking of creating a fork to do a PR but realised I am not totally
> sure how to proceed on this.
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: issues-unsubscr...@cordova.apache.org
For additional commands, e-mail: issues-h...@cordova.apache.org
