Add --throw-error (-e) flag to cli

It's common to require an exit 1 code to fail builds on CI servers. This
flag stops the need for a config file just to achieve that.

Author: simonsmith

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

@@ -47,6 +47,7 @@ Options:
   -m, --minify              minify output with cssnano
   -L, --no-lint             disable stylelint and postcss-bem-linter
   -i, --import-root [path]  the root directory for imported css files
+  -e, --throw-error         throw an error when any warnings are found
   -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

bin/suitcss

@@ -22,6 +22,7 @@ program
   .option('-m, --minify', 'minify output with cssnano')
   .option('-L, --no-lint', 'disable stylelint and postcss-bem-linter')
   .option('-i, --import-root [path]', 'the root directory for imported css files')
+  .option('-e, --throw-error', 'throw an error when any warnings are found')
   .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');
@@ -113,6 +114,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');