Refactor lint configs

.eslintrc

@@ -3,7 +3,13 @@
  "node": true
  },
  "rules": {
- "quotes": [2, "single"],
- "no-use-before-define": 0
+ "//": "single quote に統一",
+ "quotes": [2, "single", "avoid-escape"],
+
+ "//": "関数宣言文以外は hoisting 避けのため禁止",
+ "no-use-before-define": [2, "nofunc"],
+
+ "//": "この形式のキャストはくっついていないと読みづらい",
+ "space-infix-ops": [2, { "int32Hint": false }]
  }
 }

public/.eslintrc

@@ -1,8 +1,14 @@
 {
  "env": {
- "mocha": true
+ "mocha": true,
+ "browser": true,
+ "jquery": true
  },
  "globals": {
  "expect": false
+ },
+ "rules": {
+ "//": "chai の expect 構文が誤警告になるため無視",
+ "no-unused-expressions": 0
  }
 }

public/stage5/.eslintrc

@@ -1,13 +1,6 @@
 {
- "env": {
- "browser": true,
- "mocha": true
- },
  "globals": {
- "$": false,
- "jQuery": false,
  "fetch": false,
- "Promise": false,
- "HTMLCollection": false
+ "Promise": false
  }
 }

public/stage6/.eslintrc

@@ -1,17 +1,10 @@
 {
- "env": {
- "browser": true,
- "mocha": true
- },
  "rules": {
- "no-unused-expressions": 0,
+ "//": "シャレオツコードの意図が undefined 代入だとわかりやすいので許可",
  "no-undef-init": 0
  },
  "globals": {
- "$": false,
- "jQuery": false,
  "fetch": false,
- "Promise": false,
- "HTMLCollection": false
+ "Promise": false
  }
 }

public/stage7/.eslintrc

@@ -1,17 +1,16 @@
 {
- "env": {
- "browser": true,
- "mocha": true
- },
  "rules": {
- "no-unused-expressions": 0,
- "no-undef-init": 0
+ "//": "undefined 上書き問題のため、明示的な undefined 代入を許可",
+ "no-undef-init": 0,
+
+ "//": "== 演算子の振る舞いの問題の警告は誤警告",
+ "yoda": 0,
+
+ "//": "truthy, falsey を判定する問題のため、定数式を許可",
+ "no-constant-condition": 0
  },
  "globals": {
- "$": false,
- "jQuery": false,
  "fetch": false,
- "Promise": false,
- "HTMLCollection": false
+ "Promise": false
  }
 }