Merge master into develop

Author: johnboxall

CHANGELOG.md

@@ -1,4 +1,4 @@
-## To be released
+## v2.8.1 (May 8, 2017)
 - Fixes whitespace / newline management in copyright tool
 - Fix to additional space added when --update flag passed into copyright tool
 

RELEASE.md

@@ -3,3 +3,7 @@
 Use Slack to release new versions of the Mobify Code Style. 
 
 `@mobitron release mobify-code-style [minor|patch]`
+
+The release script will create the release and prep the npm package. Near the
+end it merges a release branch in `master` which kicks off `circle.yml`. The
+final npm publish step is done within the circle.yml file.

circle.yml

@@ -0,0 +1,20 @@
+version: 2
+jobs:
+  build:
+    docker:
+      - image: node:6.10.3
+    working_directory: /home/ubuntu/mobify-code-style
+    steps:
+      - checkout
+      - run: npm install
+        # npm outputs extra info on... ?linux? that breaks the diffs in docs/test/
+        # test.sh causing tests to fail on CircleCI only, but not on a macOS. As a
+        # result we use `--quiet` to suppress this npm output.
+      - run: npm test --quiet 
+      - deploy:
+          command: |
+            if [ "${CIRCLE_BRANCH}" == "master" ]; then
+              # NPM login based on env variables
+              echo -e "$NPM_USERNAME\n$NPM_PASSWORD\n$NPM_EMAIL" | npm login
+              npm publish
+            fi

docs/test/test.sh

@@ -31,6 +31,6 @@ EXIT_BAD_FILENAME=$?
 # This if will fall apart quickly as we add more test cases.
 # Need to consider some type of looping, etc possibly.
 if [ "$EXIT_GOOD" -ne "0" ] || [ "$EXIT_BAD" -ne "0" ] || [ "$EXIT_BAD_FILENAME" -ne "0" ]; then
-    echo "One or more tests failed!"
+    echo "One or more tests failed! Good: $EXIT_GOOD  Bad: $EXIT_BAD  Bad Filename: $EXIT_BAD_FILENAME"
     exit 1
 fi

package.json

@@ -1,6 +1,6 @@
 {
   "name": "mobify-code-style",
-  "version": "2.8.0",
+  "version": "2.8.1",
   "description": "Code style guide and linting tools for Mobify",
   "repository": {
     "type": "git",