chore: add "prettier" to format code and pre-commit hook to enforce (m…

…icrosoft#1049)

* add tslint-config-prettier to ts packages

* update prettier ignore and path changes

* add husky and lint-staged as dependencies

* add prettier to packages due to tslint conflicts

* update prettier npm script values

* initial run of prettier

* update prettier ignore and add lint-staged for precommit hook

* add prettier:diff build step

* move prettier ignore to local dir

* attempt to fix failing test in image

* fix tslint error

* remove forkts checker which bypassed the tslint rule overrides for prettier

.circleci/config.yml

@@ -32,6 +32,10 @@ jobs:
                 name: Bootstrap dependencies
                 command: |
                     lerna bootstrap --no-ci
+            - run: 
+                name: Ensure prettier formatting
+                command: |
+                    lerna run prettier:diff
             - run: 
                 name: Run unit tests
                 command: |

.prettierrc

@@ -0,0 +1,11 @@
+{
+    "printWidth": 90,
+    "tabWidth": 4,
+    "useTabs": false,
+    "semi": true,
+    "singleQuote": false,
+    "trailingComma": "es5",
+    "bracketSpacing": true,
+    "jsxBracketSameLine": false,
+    "arrowParens": "avoid"
+}

package.json

@@ -15,16 +15,28 @@
   "scripts": {
     "integration-tests:alpha": "node build/testing/sauce-labs/test-browsers.js alpha",
     "integration-tests:beta": "node build/testing/sauce-labs/test-browsers.js beta",
-    "integration-tests:release": "node build/testing/sauce-labs/test-browsers.js release"
+    "integration-tests:release": "node build/testing/sauce-labs/test-browsers.js release",
+    "prettier:diff": "prettier --config .prettierrc '**/*.{ts,tsx,js}' --list-different"
+  },
+  "husky": {
+    "hooks": {
+      "pre-commit": "lint-staged"
+    }
+  },
+  "lint-staged": {
+    "*.{ts,tsx,js}": ["lerna run prettier", "git add ."]
   },
   "devDependencies": {
     "@types/node": "^9.4.7",
     "dotenv": "^6.0.0",
     "eyes.selenium": "0.0.83",
     "glob": "^7.1.2",
+    "husky": "^1.1.2",
     "lerna": "^3.3.0",
+    "lint-staged": "^7.3.0",
     "markdown-it": "^8.4.1",
     "rimraf": "^2.6.2",
+    "prettier": "1.14.3",
     "ts-node": "^5.0.1",
     "typescript": "^3.0.1",
     "yargs": "^11.0.0"

packages/fast-animation/.prettierignore

@@ -0,0 +1,3 @@
+coverage/*
+dist/*
+www/*