Skip to content

Commit 118915c

Browse files
authored
Merge pull request #4542 from silamon/node18
Upgrade to Node18
2 parents 22a4dc4 + fca593d commit 118915c

File tree

6 files changed

+33
-46
lines changed

6 files changed

+33
-46
lines changed

.devcontainer/Dockerfile

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

.devcontainer/devcontainer.json

Lines changed: 17 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,20 @@
11
{
22
"name": "xterm.js",
3-
"dockerFile": "Dockerfile",
4-
"appPort": 3000,
5-
"extensions": [
6-
"dbaeumer.vscode-eslint",
7-
"editorconfig.editorconfig",
8-
"hbenl.vscode-mocha-test-adapter"
9-
]
3+
"image": "mcr.microsoft.com/devcontainers/typescript-node:0-18-buster",
4+
"features": {
5+
"ghcr.io/devcontainers/features/node:1": {} // yarn
6+
},
7+
"forwardPorts": [
8+
3000
9+
],
10+
"postCreateCommand": "yarn install",
11+
"customizations": {
12+
"vscode": {
13+
"extensions": [
14+
"dbaeumer.vscode-eslint",
15+
"editorconfig.editorconfig",
16+
"hbenl.vscode-mocha-test-adapter"
17+
]
18+
}
19+
}
1020
}

azure-pipelines.yml

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,10 +1,10 @@
11
pr:
22
branches:
3-
include: ["main", "v5"]
3+
include: ["main"]
44

55
trigger:
66
branches:
7-
include: ["main", "v5"]
7+
include: ["main"]
88

99
jobs:
1010
- job: Linux
@@ -13,7 +13,7 @@ jobs:
1313
steps:
1414
- task: NodeTool@0
1515
inputs:
16-
versionSpec: '14.x'
16+
versionSpec: '18.x'
1717
displayName: 'Install Node.js'
1818
- task: YarnInstaller@3
1919
inputs:
@@ -46,7 +46,7 @@ jobs:
4646
steps:
4747
- task: NodeTool@0
4848
inputs:
49-
versionSpec: '14.x'
49+
versionSpec: '18.x'
5050
displayName: 'Install Node.js'
5151
- task: CacheBeta@1
5252
inputs:
@@ -66,7 +66,7 @@ jobs:
6666
steps:
6767
- task: NodeTool@0
6868
inputs:
69-
versionSpec: '14.x'
69+
versionSpec: '18.x'
7070
displayName: 'Install Node.js'
7171
- task: CacheBeta@1
7272
inputs:
@@ -95,7 +95,7 @@ jobs:
9595
displayName: Install required packages
9696
- task: NodeTool@0
9797
inputs:
98-
versionSpec: '14.x'
98+
versionSpec: '18.x'
9999
displayName: 'Install Node.js'
100100
- task: YarnInstaller@3
101101
inputs:
@@ -111,11 +111,11 @@ jobs:
111111
# Integration tests are too flaky on macOS https://github.com/xtermjs/xterm.js/issues/3590
112112
# - job: macOS_IntegrationTests
113113
# pool:
114-
# vmImage: 'macOS-10.15'
114+
# vmImage: 'macOS-11'
115115
# steps:
116116
# - task: NodeTool@0
117117
# inputs:
118-
# versionSpec: '14.x'
118+
# versionSpec: '18.x'
119119
# displayName: 'Install Node.js'
120120
# - script: yarn --frozen-lockfile
121121
# displayName: 'Install dependencies and build'
@@ -132,7 +132,7 @@ jobs:
132132
steps:
133133
- task: NodeTool@0
134134
inputs:
135-
versionSpec: '14.x'
135+
versionSpec: '18.x'
136136
displayName: 'Install Node.js'
137137
- script: yarn --frozen-lockfile
138138
displayName: 'Install dependencies and build'
@@ -155,7 +155,7 @@ jobs:
155155
steps:
156156
- task: NodeTool@0
157157
inputs:
158-
versionSpec: '14.x'
158+
versionSpec: '18.x'
159159
displayName: 'Install Node.js'
160160
- task: YarnInstaller@3
161161
inputs:

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -58,7 +58,7 @@
5858
"@types/glob": "^7.2.0",
5959
"@types/jsdom": "^16.2.13",
6060
"@types/mocha": "^9.0.0",
61-
"@types/node": "^14.14.44",
61+
"@types/node": "^18.16.0",
6262
"@types/utf8": "^3.0.0",
6363
"@types/webpack": "^5.28.0",
6464
"@types/ws": "^8.2.0",

src/headless/public/Terminal.test.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ describe('Headless API Tests', function (): void {
2222

2323
it('Proposed API check', async () => {
2424
term = new Terminal({ allowProposedApi: false });
25-
throws(() => term.markers, (error) => error.message === 'You must set the allowProposedApi option to true to use proposed API');
25+
throws(() => term.markers, (error: any) => error.message === 'You must set the allowProposedApi option to true to use proposed API');
2626
});
2727

2828
it('write', async () => {

yarn.lock

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -425,10 +425,10 @@
425425
resolved "https://registry.yarnpkg.com/@types/node/-/node-12.20.50.tgz#14ba5198f1754ffd0472a2f84ab433b45ee0b65e"
426426
integrity sha512-+9axpWx2b2JCVovr7Ilgt96uc6C1zBKOQMpGtRbWT9IoR/8ue32GGMfGA4woP8QyP2gBs6GQWEVM3tCybGCxDA==
427427

428-
"@types/node@^14.14.44":
429-
version "14.14.44"
430-
resolved "https://registry.yarnpkg.com/@types/node/-/node-14.14.44.tgz#df7503e6002847b834371c004b372529f3f85215"
431-
integrity sha512-+gaugz6Oce6ZInfI/tK4Pq5wIIkJMEJUu92RB3Eu93mtj4wjjjz9EB5mLp5s1pSsLXdC/CPut/xF20ZzAQJbTA==
428+
"@types/node@^18.16.0":
429+
version "18.16.16"
430+
resolved "https://registry.yarnpkg.com/@types/node/-/node-18.16.16.tgz#3b64862856c7874ccf7439e6bab872d245c86d8e"
431+
integrity sha512-NpaM49IGQQAUlBhHMF82QH80J08os4ZmyF9MkpCzWAGuOHqE4gTEbhzd7L3l5LmWuZ6E0OiC1FweQ4tsiW35+g==
432432

433433
"@types/parse5@*":
434434
version "6.0.2"

0 commit comments

Comments
 (0)