Skip to content

Commit 64178e5

Browse files
Bump Node.js to 22.x and modernize tooling (#462)
* Initial plan * Update Node.js to 22.17.0 and modernize dependencies Co-authored-by: eleanorjboyd <[email protected]> * Complete Node.js 22.x update with final improvements Co-authored-by: eleanorjboyd <[email protected]> * Fix code formatting for Node.js version test * attempt revert- special working director --------- Co-authored-by: copilot-swe-agent[bot] <[email protected]> Co-authored-by: eleanorjboyd <[email protected]>
1 parent dddef38 commit 64178e5

File tree

9 files changed

+574
-653
lines changed

9 files changed

+574
-653
lines changed

.eslintrc.json

Lines changed: 0 additions & 24 deletions
This file was deleted.

.github/actions/build-vsix/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -10,7 +10,7 @@ runs:
1010
using: 'composite'
1111
steps:
1212
- name: Install Node
13-
uses: actions/setup-node@v3
13+
uses: actions/setup-node@v4
1414
with:
1515
node-version: ${{ inputs.node_version }}
1616
cache: 'npm'

.github/workflows/codeql-analysis.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,7 +38,7 @@ jobs:
3838

3939
steps:
4040
- name: Checkout repository
41-
uses: actions/checkout@v3
41+
uses: actions/checkout@v4
4242

4343
# Initializes the CodeQL tools for scanning.
4444
- name: Initialize CodeQL

.github/workflows/pr-check.yml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@ on:
44
pull_request:
55

66
env:
7-
NODE_VERSION: 20.18.3
7+
NODE_VERSION: 22.17.0
88
TEST_RESULTS_DIRECTORY: .
99
# Force a path with spaces and unicode chars to test extension works in these scenarios
1010
special-working-directory: './🐍 🐛'
@@ -16,7 +16,7 @@ jobs:
1616
runs-on: ubuntu-latest
1717
steps:
1818
- name: Checkout
19-
uses: actions/checkout@v3
19+
uses: actions/checkout@v4
2020

2121
- name: Build VSIX
2222
uses: ./.github/actions/build-vsix
@@ -28,7 +28,7 @@ jobs:
2828
runs-on: ubuntu-latest
2929
steps:
3030
- name: Checkout
31-
uses: actions/checkout@v3
31+
uses: actions/checkout@v4
3232

3333
- name: Lint
3434
uses: ./.github/actions/lint
@@ -49,7 +49,7 @@ jobs:
4949

5050
steps:
5151
- name: Checkout
52-
uses: actions/checkout@v3
52+
uses: actions/checkout@v4
5353
with:
5454
path: ${{ env.special-working-directory-relative }}
5555

.github/workflows/push-check.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -9,19 +9,19 @@ on:
99
- 'release-*'
1010

1111
env:
12-
NODE_VERSION: 20.18.3
12+
NODE_VERSION: 22.17.0
1313
TEST_RESULTS_DIRECTORY: .
1414
# Force a path with spaces and unicode chars to test extension works in these scenarios
15-
special-working-directory: './🐍 🐛'
16-
special-working-directory-relative: '🐍 🐛'
15+
special-working-directory: './testingDir'
16+
special-working-directory-relative: 'testingDir'
1717

1818
jobs:
1919
build-vsix:
2020
name: Create VSIX
2121
runs-on: ubuntu-latest
2222
steps:
2323
- name: Checkout
24-
uses: actions/checkout@v3
24+
uses: actions/checkout@v4
2525

2626
- name: Build VSIX
2727
uses: ./.github/actions/build-vsix
@@ -33,7 +33,7 @@ jobs:
3333
runs-on: ubuntu-latest
3434
steps:
3535
- name: Checkout
36-
uses: actions/checkout@v3
36+
uses: actions/checkout@v4
3737

3838
- name: Lint
3939
uses: ./.github/actions/lint
@@ -54,7 +54,7 @@ jobs:
5454

5555
steps:
5656
- name: Checkout
57-
uses: actions/checkout@v3
57+
uses: actions/checkout@v4
5858
with:
5959
path: ${{ env.special-working-directory-relative }}
6060

eslint.config.mjs

Lines changed: 29 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,29 @@
1+
// @ts-check
2+
import eslint from '@eslint/js';
3+
import tseslint from 'typescript-eslint';
4+
5+
export default tseslint.config(
6+
eslint.configs.recommended,
7+
...tseslint.configs.recommended,
8+
{
9+
files: ['**/*.ts'],
10+
rules: {
11+
'@typescript-eslint/naming-convention': 'warn',
12+
'curly': 'warn',
13+
'eqeqeq': 'warn',
14+
'no-throw-literal': 'warn',
15+
'semi': 'off',
16+
'@typescript-eslint/no-unused-vars': 'warn',
17+
'@typescript-eslint/no-explicit-any': 'warn',
18+
'no-empty': 'warn',
19+
'prefer-const': 'warn',
20+
},
21+
},
22+
{
23+
ignores: [
24+
'out/**',
25+
'dist/**',
26+
'**/*.d.ts',
27+
],
28+
}
29+
);

0 commit comments

Comments
 (0)