Merge branch 'master' into migrate/internal-linting

Author: MikeMcC399

CONTRIBUTING.md

@@ -5,9 +5,10 @@ Thanks for taking the time to contribute! :smile:
 To add a new rule:
   * Fork and clone this repository
   * Generate a new rule (a [yeoman generator](https://github.com/eslint/generator-eslint) is available)
-  * Run `yarn start` or `npm start`
   * Write test scenarios then implement logic
   * Describe the rule in the generated `docs` file
+  * Run `npm test` to run [Jest](https://jestjs.io/) or
+  * Run `npm start` to run [Jest](https://jestjs.io/) in [watchAll](https://jestjs.io/docs/cli#--watchall) mode where it remains active and reruns when source changes are made
   * Make sure all tests are passing
   * Add the rule to the [README](README.md) file
   * Create a PR

package.json

@@ -37,7 +37,7 @@
     "lint": "eslint \"*.js\" \"**/**/*.js\"",
     "lint-fix": "npm run lint -- --fix",
     "semantic-release": "semantic-release",
-    "start": "yarn run test-watch",
+    "start": "npm run test-watch",
     "test": "jest",
     "test-watch": "jest --watchAll",
     "prepare": "husky install"