Fixes clean-css#51 - excluding files via glob negated pattern.

Author: jakubpawlowicz

History.md

@@ -1,3 +1,8 @@
+[5.1.0 / 2021-xx-xx](https://github.com/jakubpawlowicz/clean-css-cli/compare/5.0...HEAD)
+==================
+
+* Fixed issue [#51](https://github.com/jakubpawlowicz/clean-css-cli/issues/51) - excluding files via glob negated pattern.
+
 [5.0.1 / 2021-02-11](https://github.com/jakubpawlowicz/clean-css-cli/compare/v5.0.0...v5.0.1)
 ==================
 

README.md

@@ -19,6 +19,7 @@ Previously a part of clean-css it's a separate package since clean-css 4.0.
 - [Node.js version support](#nodejs-version-support)
 - [Install](#install)
 - [Use](#use)
+  * [What's new in version 5.1](#whats-new-in-version-51)
   * [What's new in version 5.0](#whats-new-in-version-50)
   * [What's new in version 4.3](#whats-new-in-version-43)
   * [What's new in version 4.2](#whats-new-in-version-42)
@@ -60,6 +61,12 @@ Note: Global install via -g option is recommended unless you want to execute the
 cleancss -o one.min.css one.css
 ```
 
+## What's new in version 5.1
+
+clean-css-cli 5.1 will introduce the following changes / features:
+
+* accept `!path/to/file` as a way of telling `cleancss` to ignore such file, also accepts any available glob patterns.
+
 ## What's new in version 5.0
 
 clean-css-cli 5.0 introduces the following changes / features:
@@ -383,6 +390,12 @@ cleancss --batch --batch-suffix '.min' styles/*.css
 
 Remember you can use [glob matching](https://www.npmjs.com/package/glob#glob-primer) to match exactly the files you want.
 
+Since clean-css-cli 5.1 you can also use a negated pattern to exclude some files from being matched, e.g.
+
+```shell
+cleancss --batch styles/*.css !styles/*.min.css
+```
+
 ## How to specify a custom rounding precision?
 
 The level 1 `roundingPrecision` optimization option accept a string with per-unit rounding precision settings, e.g.

index.js

@@ -173,8 +173,13 @@ function findArgumentTo(option, rawArgs, args) {
 }
 
 function expandGlobs(paths) {
-  return paths.reduce(function (accumulator, path) {
-    return accumulator.concat(glob.sync(path, { nodir: true, nonull: true}));
+  var globPatterns = paths.filter(function (path) { return path[0] != '!'; });
+  var ignoredGlobPatterns = paths
+    .filter(function (path) { return path[0] == '!'; })
+    .map(function (path) { return path.substring(1); });
+
+  return globPatterns.reduce(function (accumulator, path) {
+    return accumulator.concat(glob.sync(path, { ignore: ignoredGlobPatterns, nodir: true, nonull: true }));
   }, []);
 }
 

test/binary-test.js

@@ -861,4 +861,22 @@ vows.describe('cleancss')
       }
     })
   })
+  .addBatch({
+    'batch processing with wildcard and exclude paths': binaryContext('-b ./test/fixtures-batch-5/partials/\\*\\*/*.css !./test/fixtures-batch-5/partials/one* !./test/fixtures-batch-5/partials/fiv?.css', {
+      'setup': function () {
+        execSync('cp -fr test/fixtures test/fixtures-batch-5');
+      },
+      'creates two separate minified files': function () {
+        assert.isTrue(fs.existsSync('test/fixtures-batch-5/partials/extra/four-min.css'));
+        assert.isTrue(fs.existsSync('test/fixtures-batch-5/partials/extra/three-min.css'));
+        assert.isFalse(fs.existsSync('test/fixtures-batch-5/partials/one-min.css'));
+        assert.isTrue(fs.existsSync('test/fixtures-batch-5/partials/two-min.css'));
+        assert.isTrue(fs.existsSync('test/fixtures-batch-5/partials/quoted-svg-min.css'));
+        assert.isFalse(fs.existsSync('test/fixtures-batch-5/partials/five-min.css'));
+      },
+      'teardown': function () {
+        execSync('rm -fr test/fixtures-batch-5');
+      }
+    })
+  })
   .export(module);