Merge pull request #50 from suitcss/add-throw-error-flag

Add throw error flag

Author: giuseppeg

CHANGELOG.md

@@ -1,5 +1,6 @@
 ### HEAD
 
+* Add `--throw-error` (`-e`) to CLI
 * Enable `lint` option by default
 * Add `postcss-apply`
 * Add `postcss-color-function`

README.md

@@ -43,13 +43,14 @@ Usage: suitcss [<input>] [<output>]
 Options:
 
   -h, --help                output usage information
-  -V, --version             output the version number
+  -c, --config [path]       a custom PostCSS config file
+  -i, --import-root [path]  the root directory for imported css files
+  -w, --watch               watch the input file and any imports for changes
   -m, --minify              minify output with cssnano
+  -e, --throw-error         throw an error when any warnings are found
   -L, --no-lint             disable stylelint and postcss-bem-linter
-  -i, --import-root [path]  the root directory for imported css files
-  -c, --config [path]       a custom PostCSS config file
   -v, --verbose             log verbose output for debugging
-  -w, --watch               watch the input file and any imports for changes
+  -V, --version             output the version number
 
 Examples:
 

bin/suitcss

@@ -17,14 +17,16 @@ var writeFileSync = fs.outputFileSync;
  */
 
 program
-  .version(require('../package.json').version)
   .usage('[<input>] [<output>]')
+  .option('-c, --config [path]', 'a custom PostCSS config file')
+  .option('-i, --import-root [path]', 'the root directory for imported css files')
+  .option('-w, --watch', 'watch the input file and any imports for changes')
   .option('-m, --minify', 'minify output with cssnano')
+  .option('-e, --throw-error', 'throw an error when any warnings are found')
   .option('-L, --no-lint', 'disable stylelint and postcss-bem-linter')
-  .option('-i, --import-root [path]', 'the root directory for imported css files')
-  .option('-c, --config [path]', 'a custom PostCSS config file')
   .option('-v, --verbose', 'log verbose output for debugging')
-  .option('-w, --watch', 'watch the input file and any imports for changes');
+  .version(require('../package.json').version);
+
 
 /**
  * Examples.
@@ -113,6 +115,14 @@ function run() {
       lint: program.lint
     }, config);
 
+    if (program.throwError) {
+      assign(opts, {
+        'postcss-reporter': {
+          throwError: true
+        }
+      });
+    }
+
     suitcss(css, opts).then(function(result) {
       if (output) {
         writeFileSync(output, result.css + '\n');

test/error.config.json

@@ -413,8 +413,8 @@ describe('cli', function() {
     });
   });
 
-  it('should output an error to stderr on conformance failure when throwError is set', function(done) {
-    exec('node bin/suitcss -i test/fixtures -c test/error.config.json test/fixtures/import-error.css test/fixtures/cli/output.css', function(err, stdout, stderr) {
+  it('should output an error to stderr on conformance failure when --throw-error is set', function(done) {
+    exec('node bin/suitcss -i test/fixtures -e test/fixtures/import-error.css test/fixtures/cli/output.css', function(err, stdout, stderr) {
       expect(err).to.be.an('error');
       expect(err.code).to.equal(1);
       expect(stderr).to.contain('postcss-reporter: warnings or errors were found');