Yarn & Node.js upgrade (#126)

.github/workflows/javascript-test.yml

@@ -26,7 +26,7 @@ jobs:
           key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('**/yarn.lock') }}
           restore-keys: ${{ runner.os }}-build-${{ env.cache-name }}-
       - name: Install dependencies
-        run: yarn install --frozen-lockfile
+        run: yarn install --immutable
       - run: yarn run ${{ matrix.command }}
 
   conclude:

.gitignore

@@ -1,6 +1,7 @@
 /node_modules
 /dist*
 .env
+.yarn/install-state.gz
 
 # Editor directories and files
 .vscode/*

.yarnrc.yml

@@ -0,0 +1,3 @@
+nodeLinker: node-modules
+
+yarnPath: .yarn/releases/yarn-4.3.0.cjs

Dockerfile

@@ -1,16 +1,17 @@
-FROM node:20 as Builder
+FROM node:22 as Builder
 
 WORKDIR /action
 
-COPY package.json yarn.lock ./
+COPY .yarn/ ./.yarn/
+COPY package.json yarn.lock .yarnrc.yml ./
 
-RUN yarn install --frozen-lockfile
+RUN yarn install --immutable
 
 COPY . .
 
 RUN yarn run build
 
-FROM node:20-slim
+FROM node:22-slim
 
 COPY --from=Builder /action/dist /action
 

package.json

@@ -8,11 +8,11 @@
     "build": "ncc build --license LICENSE",
     "cli": "ncc build src/cli.ts -o dist-cli && node dist-cli",
     "test": "jest",
-    "fix": "eslint --fix '{src,test}/**/*'",
-    "lint": "eslint '{src,test}/**/*'"
+    "fix": "npx eslint --fix 'src/**/*.ts' && npx prettier --write 'src/**/*.{ts,yml}'",
+    "lint": "npx eslint 'src/**/*.ts' && npx prettier --check 'src/**/*.{ts,yml}'"
   },
   "engines": {
-    "node": ">=20.0.0"
+    "node": ">=22.0.0"
   },
   "repository": {
     "type": "git",
@@ -25,21 +25,22 @@
   },
   "homepage": "https://github.com/paritytech/review-bot#readme",
   "devDependencies": {
-    "@eng-automation/js-style": "^2.3.0",
+    "@eng-automation/js-style": "^3.1.0",
     "@octokit/webhooks-types": "^7.3.1",
     "@types/jest": "^29.5.11",
     "@vercel/ncc": "^0.38.1",
     "jest": "^29.7.0",
     "jest-mock-extended": "^3.0.5",
     "ts-jest": "^29.1.2",
-    "typescript": "^5.3.3"
+    "typescript": "^5.4.5"
   },
   "dependencies": {
     "@actions/core": "^1.10.1",
     "@actions/github": "^6.0.0",
-    "@eng-automation/js": "^1.0.2",
-    "@polkadot/api": "^10.13.1",
-    "joi": "^17.6.4",
+    "@eng-automation/js": "^2.2.0",
+    "@polkadot/api": "^11.3.1",
+    "joi": "^17.13.1",
     "yaml": "^2.3.4"
-  }
+  },
+  "packageManager": "[email protected]"
 }

src/test/config.yml

@@ -1,14 +1,14 @@
 rules:
   - name: CI files
     condition:
-      include: 
+      include:
         - ^\.gitlab-ci\.yml
         - ^docker/.*
         - ^\.github/.*
         - ^\.gitlab/.*
         - ^\.config/nextest.toml
         - ^\.cargo/.*
-      exclude: 
+      exclude:
         - ^./gitlab/pipeline/zombienet.yml$
     minApprovals: 2
     type: basic
@@ -19,12 +19,12 @@ rules:
   - name: Audit rules
     type: basic
     condition:
-      include: 
+      include:
         - ^polkadot/runtime\/(kusama|polkadot|common)\/.*
         - ^polkadot/primitives/src\/.+\.rs$
         - ^substrate/primitives/.*
         - ^substrate/frame/.*
-      exclude: 
+      exclude:
         - ^polkadot/runtime\/(kusama|polkadot)\/src\/weights\/.+\.rs$
         - ^substrate\/frame\/.+\.md$
     minApprovals: 2
@@ -77,7 +77,7 @@ rules:
   # if there are any changes in the bridges subtree (in case of backport changes back to bridges repo)
   - name: Bridges subtree files
     type: basic
-    condition: 
+    condition:
       include:
         - ^cumulus/bridges/.*
     minApprovals: 1
@@ -88,7 +88,7 @@ rules:
 
   - name: FRAME coders substrate
     condition:
-      include: 
+      include:
         - ^substrate/frame/(?!.*(nfts/.*|uniques/.*|babe/.*|grandpa/.*|beefy|merkle-mountain-range/.*|contracts/.*|election|nomination-pools/.*|staking/.*|aura/.*))
     type: "and"
     reviewers: