@semantic-release/github 5.0.0 → 5.0.1

package/lib/glob-assets.js

@@ -1,8 +1,13 @@
+const path = require('path');
 const {basename} = require('path');
-const {isPlainObject, castArray, uniqWith} = require('lodash');
+const {isPlainObject, castArray, uniqWith, uniq} = require('lodash');
+const dirGlob = require('dir-glob');
 const globby = require('globby');
 const debug = require('debug')('semantic-release:github');
 
+const filesTransform = (files, cwd, transform) =>
+  files.map(file => `${file.startsWith('!') ? '!' : ''}${transform(cwd, file.startsWith('!') ? file.slice(1) : file)}`);
+
 module.exports = async ({cwd}, assets) =>
   uniqWith(
     []
@@ -10,16 +15,30 @@ module.exports = async ({cwd}, assets) =>
         ...(await Promise.all(
           assets.map(async asset => {
             // Wrap single glob definition in Array
-            const glob = castArray(isPlainObject(asset) ? asset.path : asset);
+            let glob = castArray(isPlainObject(asset) ? asset.path : asset);
+            // TODO Temporary workaround for https://github.com/kevva/dir-glob/issues/7 and https://github.com/mrmlnc/fast-glob/issues/47
+            glob = uniq([
+              ...filesTransform(await dirGlob(filesTransform(glob, cwd, path.resolve)), cwd, path.relative),
+              ...glob,
+            ]);
+
             // Skip solo negated pattern (avoid to include every non js file with `!**/*.js`)
             if (glob.length <= 1 && glob[0].startsWith('!')) {
               debug(
-                'skipping the negated glob %o as its alone in its group and would retrieve a large amount of files ',
+                'skipping the negated glob %o as its alone in its group and would retrieve a large amount of files',
                 glob[0]
               );
               return [];
             }
-            const globbed = await globby(glob, {cwd, expandDirectories: true, gitignore: false, dot: true});
+
+            const globbed = await globby(glob, {
+              cwd,
+              expandDirectories: true,
+              gitignore: false,
+              dot: true,
+              onlyFiles: false,
+            });
+
             if (isPlainObject(asset)) {
               if (globbed.length > 1) {
                 // If asset is an Object with a glob the `path` property that resolve to multiple files,
@@ -34,10 +53,12 @@ module.exports = async ({cwd}, assets) =>
               // - other properties of the original asset definition
               return {...asset, path: globbed[0] || asset.path};
             }
+
             if (globbed.length > 0) {
               // If asset is a String definition, output each files matched
               return globbed;
             }
+
             // If asset is a String definition but no match is found, output the elements of the original glob (each one will be considered as a missing file)
             return glob;
           })

package/package.json

@@ -1,7 +1,7 @@
 {
   "name": "@semantic-release/github",
   "description": "Set of semantic-release plugins for publishing a GitHub release",
-  "version": "5.0.0",
+  "version": "5.0.1",
   "author": "Pierre Vanduynslager (https://twitter.com/@pvdlg_)",
   "bugs": {
     "url": "https://github.com/semantic-release/github/issues"
@@ -21,6 +21,7 @@
     "aggregate-error": "^1.0.0",
     "bottleneck": "^2.0.1",
     "debug": "^3.1.0",
+    "dir-glob": "^2.0.0",
     "fs-extra": "^7.0.0",
     "globby": "^8.0.0",
     "http-proxy-agent": "^2.1.0",