Move custom ESLint rules into internal package (#2498)

Based on facebook/react@30cce21#diff-a7adbbe27ccec89b583ac77c28b521ac

Author: IvanGoncharov

.eslintrc.yml

@@ -6,10 +6,18 @@ env:
   node: true
 reportUnusedDisableDirectives: true
 plugins:
+  - graphql-internal
   - flowtype
   - import
 
 rules:
+  ##############################################################################
+  # Internal rules located in 'resources/eslint-rules'.
+  # See './resources/eslint-rules/README.md'
+  ##############################################################################
+
+  graphql-internal/no-dir-import: error
+
   ##############################################################################
   # `eslint-plugin-flowtype` rule list based on `v4.6.x`
   # https://github.com/gajus/eslint-plugin-flowtype#eslint-plugin-flowtype

package.json

@@ -28,7 +28,7 @@
     "test:ci": "yarn check --integrity && npm run prettier:check && npm run lint -- --no-cache && npm run check && npm run testonly:cover && npm run check:ts && npm run check:spelling && npm run build",
     "testonly": "mocha --full-trace src/**/__tests__/**/*-test.js",
     "testonly:cover": "nyc npm run testonly",
-    "lint": "eslint --rulesdir './resources/eslint-rules' --rule 'no-dir-import: error' --cache --ext .js,.ts src resources",
+    "lint": "eslint --cache --ext .js,.ts src resources",
     "benchmark": "node --noconcurrent_sweeping --expose-gc --predictable ./resources/benchmark.js",
     "prettier": "prettier --ignore-path .gitignore --write --list-different \"**/*.{js,ts,md,json,yml}\"",
     "prettier:check": "prettier --ignore-path .gitignore --check \"**/*.{js,ts,md,json,yml}\"",
@@ -56,6 +56,7 @@
     "dtslint": "3.3.0",
     "eslint": "6.8.0",
     "eslint-plugin-flowtype": "4.6.0",
+    "eslint-plugin-graphql-internal": "link:./resources/eslint-rules",
     "eslint-plugin-import": "2.20.1",
     "flow-bin": "0.120.1",
     "mocha": "7.1.0",

resources/eslint-rules/README.md

@@ -0,0 +1,6 @@
+# Custom ESLint Rules
+
+This is a dummy npm package that allows us to treat it as an `eslint-plugin-graphql-internal`.
+It's not actually published, nor are the rules here useful for users of graphql.
+
+**If you modify this rule, you must re-run `yarn` for it to take effect.**

resources/eslint-rules/index.js

@@ -0,0 +1,9 @@
+// @noflow
+
+'use strict';
+
+module.exports = {
+  rules: {
+    'no-dir-import': require('./no-dir-import'),
+  },
+};

resources/eslint-rules/package.json

@@ -0,0 +1,4 @@
+{
+  "name": "eslint-plugin-graphql-internal",
+  "version": "0.0.0"
+}

yarn.lock

@@ -1728,6 +1728,10 @@ [email protected]:
   dependencies:
     lodash "^4.17.15"
 
+"eslint-plugin-graphql-internal@link:./resources/eslint-rules":
+  version "0.0.0"
+  uid ""
+
 [email protected]:
   version "2.20.1"
   resolved "https://registry.yarnpkg.com/eslint-plugin-import/-/eslint-plugin-import-2.20.1.tgz#802423196dcb11d9ce8435a5fc02a6d3b46939b3"