chore: dependency upgrades, import order, and no one-line if statements (#19)

* Enforce curly braces for all if statements * Upgrades * import order configuration

Author: rickharris

index.js

@@ -7,8 +7,43 @@ module.exports = {
  // This doesn't work for us because we use `inject-loader` for testing JS
  // files, and can't be automatically configured in webpack.config.js.
  "import/no-webpack-loader-syntax": "off",
- "import/order": "warn",
+ "import/order": [
+ "error",
+ {
+ alphabetize: {
+ order: "asc"
+ },
+ pathGroups: [
+ {
+ pattern: "abstract-di/**",
+ group: "external",
+ position: "after"
+ },
+ {
+ pattern: "core/**",
+ group: "external",
+ position: "after"
+ },
+ {
+ pattern: "desktop/**",
+ group: "external",
+ position: "after"
+ },
+ {
+ pattern: "support/**",
+ group: "external",
+ position: "after"
+ },
+ {
+ pattern: "web/**",
+ group: "external",
+ position: "after"
+ }
+ ]
+ }
+ ],
  "mocha/no-exclusive-tests": "error",
+ curly: ["error", "all"],
  // Prettier automatically uses the least amount of parens possible, so this
  // does more harm than good.
  "no-mixed-operators": "off",

package.json

@@ -11,29 +11,33 @@
  "lint": "eslint ."
  },
  "devDependencies": {
- "babel-eslint": "9.x",
- "eslint": "5.x",
- "eslint-config-react-app": "^3.0.5",
- "eslint-plugin-flowtype": "2.x",
- "eslint-plugin-import": "2.x",
- "eslint-plugin-jsx-a11y": "6.x",
+ "@typescript-eslint/eslint-plugin": "^2.15.0",
+ "@typescript-eslint/parser": "^2.15.0",
+ "babel-eslint": "^10.0.3",
+ "eslint": "^6.8.0",
+ "eslint-config-react-app": "^5.1.0",
+ "eslint-plugin-flowtype": "^3.13.0",
+ "eslint-plugin-import": "^2.19.1",
+ "eslint-plugin-jsx-a11y": "^6.2.3",
  "eslint-plugin-mocha": "^5.2.0",
  "eslint-plugin-prettier": "^3.0.0",
- "eslint-plugin-react": "7.x",
- "eslint-plugin-react-hooks": "^2.0.1",
- "prettier": "1.15.3"
+ "eslint-plugin-react": "^7.17.0",
+ "eslint-plugin-react-hooks": "^1.7.0",
+ "prettier": "1.19.1"
  },
  "peerDependencies": {
- "babel-eslint": "9.x",
- "eslint": "5.x",
- "eslint-config-react-app": "^3.0.5",
- "eslint-plugin-flowtype": "2.x",
- "eslint-plugin-import": "2.x",
- "eslint-plugin-jsx-a11y": "6.x",
+ "@typescript-eslint/eslint-plugin": "^2.15.0",
+ "@typescript-eslint/parser": "^2.15.0",
+ "babel-eslint": "^10.0.3",
+ "eslint": "^6.8.0",
+ "eslint-config-react-app": "^5.1.0",
+ "eslint-plugin-flowtype": "^3.13.0",
+ "eslint-plugin-import": "^2.19.1",
+ "eslint-plugin-jsx-a11y": "^6.2.3",
  "eslint-plugin-mocha": "^5.2.0",
  "eslint-plugin-prettier": "^3.0.0",
- "eslint-plugin-react": "7.x",
- "eslint-plugin-react-hooks": "^2.0.1",
- "prettier": "1.15.3"
+ "eslint-plugin-react": "^7.17.0",
+ "eslint-plugin-react-hooks": "^1.7.0",
+ "prettier": "1.19.1"
  }
 }