cordova-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From audreyso <...@git.apache.org>
Subject [GitHub] cordova-lib pull request #602: CB-12774 : Don't munge scoped plugin IDs anym...
Date Wed, 25 Oct 2017 23:19:59 GMT
Github user audreyso commented on a diff in the pull request:

    https://github.com/apache/cordova-lib/pull/602#discussion_r147012615
  
    --- Diff: src/cordova/util.js ---
    @@ -255,11 +255,20 @@ function findPlugins (pluginDir) {
         var plugins = [];
     
         if (fs.existsSync(pluginDir)) {
    -        plugins = fs.readdirSync(pluginDir).filter(function (fileName) {
    -            var pluginPath = path.join(pluginDir, fileName);
    -            var isPlugin = isDirectory(pluginPath) || isSymbolicLink(pluginPath);
    -            return fileName !== '.svn' && fileName !== 'CVS' && isPlugin;
    -        });
    +        plugins = fs.readdirSync(pluginDir)
    +            .reduce(function (plugins, pluginOrScope) {
    +                if (pluginOrScope[0] === '@') {
    +                    plugins.push(...fs.readdirSync(path.join(pluginDir, pluginOrScope)).map(s
=> path.join(pluginOrScope, s)));
    --- End diff --
    
    Hi! This line will need to be changed as it will not work with Node 4, which is still
currently supported. http://node.green/#ES2015-syntax-spread-------operator-with-generic-iterables--in-calls


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@cordova.apache.org
For additional commands, e-mail: dev-help@cordova.apache.org


Mime
View raw message