Merge pull request #159 from PHILLIPS71/chore/update-ci-workflow-actions

chore: update ci workflow actions

Author: captbaritone

.github/workflows/ci.yml

@@ -15,8 +15,8 @@ jobs:
       matrix:
         node-version: [18.x, 20.x, 22.x]
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
           cache: 'yarn'
@@ -29,8 +29,8 @@ jobs:
     name: Lint
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: 22.x
           cache: 'yarn'
@@ -47,8 +47,8 @@ jobs:
     if: github.event_name == 'push' && github.repository == 'relayjs/eslint-plugin-relay'
     needs: [build, lint]
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions/setup-node@v2
+      - uses: actions/checkout@v4
+      - uses: actions/setup-node@v4
         with:
           node-version: 22.x
           registry-url: https://registry.npmjs.org/

README.md

@@ -52,10 +52,12 @@ The following rules support suppression within graphql tags:
 Supported rules can be suppressed by adding `# eslint-disable-next-line relay/name-of-rule` to the preceding line:
 
 ```js
-graphql`fragment foo on Page {
-  # eslint-disable-next-line relay/must-colocate-fragment-spreads
-  ...unused1
-}`
+graphql`
+  fragment foo on Page {
+    # eslint-disable-next-line relay/must-colocate-fragment-spreads
+    ...unused1
+  }
+`;
 ```
 
 Note that only the `eslint-disable-next-line` form of suppression works. `eslint-disable-line` doesn't currently work until graphql-js provides support for [parsing Comment nodes](https://github.com/graphql/graphql-js/issues/2241) in their AST.

package.json

@@ -32,5 +32,6 @@
     "mocha": "^9.1.3",
     "prettier": "^2.4.1",
     "typescript": "^5.8.3"
-  }
+  },
+  "packageManager": "[email protected]+sha1.ac34549e6aa8e7ead463a7407e1c7390f61a6610"
 }

src/rule-generated-typescript-types.js

@@ -351,7 +351,9 @@ module.exports = {
       }
       if (arg.type === 'Identifier') {
         const name = arg.name;
-        let scope = sourceCode.getScope ? sourceCode.getScope(arg) : context.getScope();
+        let scope = sourceCode.getScope
+          ? sourceCode.getScope(arg)
+          : context.getScope();
         while (scope != null) {
           for (const variable of scope.variables) {
             if (variable.name === name) {

src/utils.js

@@ -45,7 +45,10 @@ function getLoc(context, templateNode, graphQLNode) {
   const start = startAndEnd[0];
   const end = startAndEnd[1];
   return {
-    start: getLocFromIndex(context.sourceCode ?? context.getSourceCode(), start),
+    start: getLocFromIndex(
+      context.sourceCode ?? context.getSourceCode(),
+      start
+    ),
     end: getLocFromIndex(context.sourceCode ?? context.getSourceCode(), end)
   };
 }
@@ -105,7 +108,9 @@ function isGraphQLTag(tag) {
 }
 
 function shouldLint(context) {
-  return /graphql|relay/i.test((context.sourceCode ?? context.getSourceCode()).text);
+  return /graphql|relay/i.test(
+    (context.sourceCode ?? context.getSourceCode()).text
+  );
 }
 
 function hasPrecedingEslintDisableComment(node, commentText) {

test/compat-uses-vars.js

@@ -7,7 +7,7 @@
 
 'use strict';
 
-const { builtinRules } = require('eslint/use-at-your-own-risk');
+const {builtinRules} = require('eslint/use-at-your-own-risk');
 const RuleTester = require('eslint').RuleTester;
 const rules = require('..').rules;
 
@@ -16,7 +16,7 @@ const ruleTester = new RuleTester({
   plugins: {
     relay: {
       rules: {
-        'compat-uses-vars': rules['compat-uses-vars'],
+        'compat-uses-vars': rules['compat-uses-vars']
       }
     }
   }

test/function-required-argument.js

@@ -11,7 +11,10 @@ const rules = require('..').rules;
 const RuleTester = require('eslint').RuleTester;
 
 const ruleTester = new RuleTester({
-  languageOptions: {ecmaVersion: 6, parser: require('@typescript-eslint/parser')}
+  languageOptions: {
+    ecmaVersion: 6,
+    parser: require('@typescript-eslint/parser')
+  }
 });
 
 ruleTester.run(

test/future-added-value.js

@@ -17,8 +17,11 @@ const ruleTester = new RuleTester({
     ecmaVersion: 6,
     sourceType: 'module',
     parser: require('@babel/eslint-parser'),
-    parserOptions: { requireConfigFile: false, babelOptions: { "presets": ["@babel/preset-flow"] } }
-  },
+    parserOptions: {
+      requireConfigFile: false,
+      babelOptions: {presets: ['@babel/preset-flow']}
+    }
+  }
 });
 
 const FUTURE_ADDED_VALUE_MESSAGE =

test/generated-typescript-types.js

@@ -11,7 +11,11 @@ const rules = require('..').rules;
 const RuleTester = require('eslint').RuleTester;
 
 const ruleTester = new RuleTester({
-  languageOptions: {ecmaVersion: 6, parser: require('@typescript-eslint/parser'), parserOptions: {ecmaFeatures: {jsx: true}}},
+  languageOptions: {
+    ecmaVersion: 6,
+    parser: require('@typescript-eslint/parser'),
+    parserOptions: {ecmaFeatures: {jsx: true}}
+  }
 });
 
 const HAS_ESLINT_BEEN_UPGRADED_YET = false;

test/hook-required-argument.js

@@ -11,7 +11,10 @@ const rules = require('..').rules;
 const RuleTester = require('eslint').RuleTester;
 
 const ruleTester = new RuleTester({
-  languageOptions: {ecmaVersion: 6, parser: require('@typescript-eslint/parser')}
+  languageOptions: {
+    ecmaVersion: 6,
+    parser: require('@typescript-eslint/parser')
+  }
 });
 
 ruleTester.run('hook-required-argument', rules['hook-required-argument'], {