Linting

Author: benjie

.eslintignore

@@ -0,0 +1 @@
+node_modules

.eslintrc.js

@@ -1,23 +1,16 @@
 module.exports = {
   parser: "babel-eslint",
-  extends: ["airbnb", "prettier"],
-  plugins: ["jest", "prettier"],
+  extends: ["eslint:recommended", "prettier"],
+  plugins: ["jest"],
   env: {
     jest: true,
-    node: true
+    node: true,
+    es6: true,
   },
   rules: {
     // Autofix removes debugger automatically, which makes debugging annoying.
     "no-debugger": 0,
 
-    // Prettier
-    "prettier/prettier": [
-      "error",
-      {
-        trailingComma: "es5"
-      }
-    ],
-
     // Jest
     "jest/no-focused-tests": 2,
     "jest/no-identical-title": 2,
@@ -33,30 +26,30 @@ module.exports = {
       2,
       {
         argsIgnorePattern: "^_",
-        varsIgnorePattern: "^_"
-      }
+        varsIgnorePattern: "^_",
+      },
     ],
     "no-cond-assign": [2, "except-parens"],
     "no-unused-expressions": [
       0,
       {
-        allowTernary: true
-      }
+        allowTernary: true,
+      },
     ],
     "prefer-arrow-callback": [
       "error",
       {
-        allowNamedFunctions: true
-      }
+        allowNamedFunctions: true,
+      },
     ],
     "no-param-reassign": [
       "error",
       {
         props: true,
         // Allow overwriting properties on 'memo' which is the name we tend to use in `.reduce(...)` calls
-        ignorePropertyModificationsFor: ["memo"]
-      }
+        ignorePropertyModificationsFor: ["memo"],
+      },
     ],
-    camelcase: 0
-  }
+    camelcase: 0,
+  },
 };

.gitignore

@@ -0,0 +1 @@
+node_modules/

.prettierrc.js

@@ -0,0 +1,4 @@
+module.exports = {
+  trailingComma: "all",
+  proseWrap: "always",
+};

gatsby-node.js

@@ -11,7 +11,7 @@ exports.sourceNodes = async (
     fieldName = "postgres",
     refetchInterval,
     ...options
-  }
+  },
 ) => {
   const { connectionString, schema: postgresSchema, ...rest } = options;
 

lib/PostGraphileLink.js

@@ -16,7 +16,7 @@ module.exports = class PostGraphileLink extends ApolloLink {
         this.schema,
         operation.query,
         operation.variables,
-        operation.operationName
+        operation.operationName,
       )
         .then(data => {
           if (!observer.closed) {

lib/RenamedQueryPlugin.js

@@ -5,8 +5,8 @@ module.exports = function RenamedQueryPlugin(builder) {
       {
         $$isQuery: Symbol("isQuery"),
       },
-      `Extending Build`
-    )
+      `Extending Build`,
+    ),
   );
   builder.hook("GraphQLSchema", (schema, build) => {
     const {
@@ -28,14 +28,14 @@ module.exports = function RenamedQueryPlugin(builder) {
       {
         __origin: `gatsby-source-pg override`,
         isRootQuery: true,
-      }
+      },
     );
     return extend(
       schema,
       {
         query: queryType,
       },
-      `Adding 'query' type to Schema`
+      `Adding 'query' type to Schema`,
     );
   });
 };

lib/createSchema.js

@@ -9,7 +9,7 @@ function requireFrom(modules, moduleName) {
     if (modules.length > 1) {
       const result = requireFrom(
         modules.slice(0, modules.length - 1),
-        moduleName
+        moduleName,
       );
       if (result) return result;
     }
@@ -21,7 +21,7 @@ function requireFrom(modules, moduleName) {
 }
 const graphileBuild = requireFrom(
   ["postgraphile", "postgraphile-core"],
-  "graphile-build"
+  "graphile-build",
 );
 
 const RenamedQueryPlugin = require("./RenamedQueryPlugin");

lib/performQuery.js

@@ -6,7 +6,7 @@ module.exports = async function performQuery(
   schema,
   query,
   variables,
-  operationName
+  operationName,
 ) {
   const queryString = typeof query === "string" ? query : print(query);
   return withPostGraphileContext({ pgPool }, async context =>
@@ -16,7 +16,7 @@ module.exports = async function performQuery(
       null,
       { ...context }, // You can add more to context if you like
       variables,
-      operationName
-    )
+      operationName,
+    ),
   );
 };

package.json

@@ -4,6 +4,9 @@
   "description": "PostgreSQL source for Gatsby, powered by PostGraphile",
   "main": "index.js",
   "scripts": {
+    "lint:fix": "yarn prettier --write && yarn eslint --fix",
+    "prettier": "prettier --ignore-path .eslintignore '**/*.{js,jsx,ts,tsx,graphql,md,yaml,yml}'",
+    "eslint": "eslint --ext js,jsx,ts,tsx .",
     "test": "jest"
   },
   "repository": {
@@ -43,12 +46,12 @@
   },
   "devDependencies": {
     "babel-eslint": "^10.0.1",
-    "eslint": "^5.8.0",
+    "eslint": "^6.8.0",
     "eslint-config-airbnb": "^17.1.0",
-    "eslint-config-prettier": "^3.1.0",
+    "eslint-config-prettier": "^6.10.1",
     "eslint-plugin-graphql": "benjie/eslint-plugin-graphql#ignore-fragments-built",
     "eslint-plugin-import": "^2.14.0",
-    "eslint-plugin-jest": "^21.26.2",
+    "eslint-plugin-jest": "^23.8.2",
     "eslint-plugin-jsx-a11y": "^6.1.2",
     "eslint-plugin-prettier": "^3.0.0",
     "eslint-plugin-react": "^7.11.1",