Merge pull request #452 from tschaub/updates

Assorted updates

Author: tschaub

.github/dependabot.yml

@@ -0,0 +1,14 @@
+version: 2
+
+updates:
+  - package-ecosystem: npm
+    directory: "/"
+    schedule:
+      interval: weekly
+    open-pull-requests-limit: 10
+    versioning-strategy: increase-if-necessary
+
+  - package-ecosystem: github-actions
+    directory: "/"
+    schedule:
+      interval: weekly

.github/workflows/test.yml

@@ -22,9 +22,9 @@ jobs:
         os:
           - ubuntu-latest
         node:
-          - 12
           - 14
           - 16
+          - 18
 
     steps:
       - name: Clone repository

lib/index.js

@@ -53,7 +53,7 @@ exports.defaults = {
 /**
  * Push a git branch to a remote (pushes gh-pages by default).
  * @param {string} basePath The base path.
- * @param {Object} config Publish options.
+ * @param {object} config Publish options.
  * @param {Function} callback Callback.
  * @return {Promise} A promise.
  */
@@ -88,12 +88,13 @@ exports.publish = function publish(basePath, config, callback) {
 
   try {
     if (!fs.statSync(basePath).isDirectory()) {
-      done(new Error('The "base" option must be an existing directory'));
-      return;
+      const err = new Error('The "base" option must be an existing directory');
+      done(err);
+      return Promise.reject(err);
     }
   } catch (err) {
     done(err);
-    return;
+    return Promise.reject(err);
   }
 
   const files = globby

lib/util.js

@@ -9,17 +9,17 @@ const fs = require('fs-extra');
  * @return {Array<string>} List of directory paths.
  */
 function uniqueDirs(files) {
-  const dirs = {};
+  const dirs = new Set();
   files.forEach((filepath) => {
     const parts = path.dirname(filepath).split(path.sep);
     let partial = parts[0] || '/';
-    dirs[partial] = true;
+    dirs.add(partial);
     for (let i = 1, ii = parts.length; i < ii; ++i) {
       partial = path.join(partial, parts[i]);
-      dirs[partial] = true;
+      dirs.add(partial);
     }
   });
-  return Object.keys(dirs);
+  return Array.from(dirs);
 }
 exports.uniqueDirs = uniqueDirs;
 
@@ -70,7 +70,7 @@ exports.dirsToCreate = dirsToCreate;
 
 /**
  * Copy a file.
- * @param {Object} obj Object with src and dest properties.
+ * @param {object} obj Object with src and dest properties.
  * @param {function(Error)} callback Callback
  */
 function copyFile(obj, callback) {