Merge pull request #18 from Agoric/standardize-eslint-rules

standardize eslint rules and eslint npm scripts

.circleci/config.yml

@@ -20,7 +20,7 @@ jobs:
             - ./node_modules
       - run:
           name: Lint
-          command: npm run lint
+          command: npm run lint-check
       - run:
           name: Test
           command: npm test

.eslintrc.js

@@ -1,14 +1,17 @@
 module.exports = {
   extends: ['airbnb', 'plugin:prettier/recommended'],
   env: {
-    es6: true,
-    mocha: true,
+    es6: true, // supports new ES6 globals (e.g., new types such as Set)
   },
   rules: {
     'implicit-arrow-linebreak': 'off',
     'function-paren-newline': 'off',
     'arrow-parens': 'off',
     strict: 'off',
     'no-console': 'off',
+    'no-unused-vars': ['error', { argsIgnorePattern: '^_' }],
+    'no-return-assign': 'off',
+    'no-param-reassign': 'off',
+    'no-restricted-syntax': ['off', 'ForOfStatement'],
   },
 };

.npmignore

@@ -2,6 +2,6 @@
 test
 
 # Settings
-.travis.yml
+.circleci
 .eslintrc.js
 .prettierrc.json

package.json

@@ -6,9 +6,9 @@
   "module": "index.js",
   "scripts": {
     "test": "node test/test.js",
-    "format": "prettier --write '**/*.{js,jsx}'",
-    "lint": "eslint '**/*.{js,jsx}'",
-    "create-cjs": "rollup index.js --file index-commonjs.js --format cjs"
+    "create-cjs": "rollup index.js --file index-commonjs.js --format cjs",
+    "lint-fix": "eslint --fix '**/*.{js,jsx}'",
+    "lint-check": "eslint '**/*.{js,jsx}'"
   },
   "repository": {
     "type": "git",