Merge pull request #64 from actions/eslint-roller

Update ESLint config and expand usage

.eslintignore

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

.eslintrc.json

@@ -1,19 +1,17 @@
 {
   "env": {
     "commonjs": true,
-    "es6": true,
+    "es2021": true,
-    "jest": true,
+    "node": true,
-    "node": true
+    "jest": true
-  },
-  "extends": "eslint:recommended",
-  "globals": {
-    "Atomics": "readonly",
-    "SharedArrayBuffer": "readonly"
   },
+  "plugins": ["github"],
+  "extends": ["eslint:recommended", "prettier", "plugin:github/internal"],
   "parserOptions": {
-    "ecmaVersion": 2020
+    "ecmaVersion": 12
   },
   "rules": {
     "semi": ["error", "never"]
-  }
+  },
+  "ignorePatterns": ["/dist/", "/pre/"]
 }

.github/workflows/check-linter.yml

@@ -1,4 +1,4 @@
-name: Checking linter
+name: Check linter
 
 on:
   push:
@@ -31,4 +31,4 @@ jobs:
         run: npm ci
 
       - name: Verify linter
-        run: npm run lint
+        run: npm run lint:check

package.json

@@ -23,12 +23,13 @@
     "prettier": "^2.4.1"
   },
   "scripts": {
-    "lint": "eslint src",
+    "all": "npm run format && npm run lint && npm run prepare && npm run test",
-    "prepare": "ncc build src/index.js -o dist --source-map --license licenses.txt && ncc build src/pre.js -o pre --source-map --license licenses.txt",
-    "test": "jest",
     "format": "prettier --write .",
     "format:check": "prettier --check .",
-    "all": "npm run format && npm run lint && npm run prepare && npm run test"
+    "lint": "DEBUG=eslint:cli-engine eslint --fix .",
+    "lint:check": "DEBUG=eslint:cli-engine eslint .",
+    "prepare": "ncc build src/index.js -o dist --source-map --license licenses.txt && ncc build src/pre.js -o pre --source-map --license licenses.txt",
+    "test": "jest"
   },
   "repository": {
     "type": "git",

src/index.test.js

@@ -5,8 +5,6 @@ const path = require('path')
 const nock = require('nock')
 const axios = require('axios')
 
-const { expect, jest } = require('@jest/globals')
-
 const { Deployment } = require('./deployment')
 
 describe('with all environment variables set', () => {

src/pre.test.js

@@ -2,8 +2,6 @@ const core = require('@actions/core')
 const process = require('process')
 const axios = require('axios')
 
-const { expect, jest } = require('@jest/globals')
-
 const { emitTelemetry } = require('./pre')
 
 describe('emitTelemetry', () => {