Skip to content

Commit

Permalink
chore: use Node.js 20 in CI VSCODE-586 (#770)
Browse files Browse the repository at this point in the history
* chore: use Node.js 20 in CI VSCODE-586

* fix: try to resolve a regression of out of memory crashes of webpack
  • Loading branch information
alenakhineika authored Aug 12, 2024
1 parent 1fc7714 commit 95aa3a3
Show file tree
Hide file tree
Showing 5 changed files with 10 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/actions/test-and-build/action.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,7 +76,7 @@ runs:

- name: Build .vsix
env:
NODE_OPTIONS: "--require ./scripts/no-npm-list-fail.js"
NODE_OPTIONS: "--require ./scripts/no-npm-list-fail.js --max_old_space_size=4096"
# NOTE: --githubBranch is "The GitHub branch used to infer relative links in README.md."
run: |
npx vsce package --githubBranch main
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/draft-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,7 @@ jobs:
- name: Setup Node.js Environment
uses: actions/setup-node@v3
with:
node-version: 16.x
node-version: 20.16.0

- name: Determine Next Version
shell: bash
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/publish-release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ jobs:
- name: Setup Node.js Environment
uses: actions/setup-node@v3
with:
node-version: 16.x
node-version: 20.16.0

- name: Install npm
run: npm install -g [email protected]
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/test-and-build.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ jobs:
- name: Setup Node.js Environment
uses: actions/setup-node@v3
with:
node-version: 16.x
node-version: 20.16.0

- name: Run tests and build
uses: ./.github/workflows/actions/test-and-build
Expand Down
6 changes: 6 additions & 0 deletions webpack.config.js
Original file line number Diff line number Diff line change
Expand Up @@ -201,6 +201,12 @@ module.exports = (env, argv) => {
process: 'process/browser',
}),
],
watchOptions: {
// For some systems, watching many files can result in a lot of CPU or memory usage
// https://webpack.js.org/configuration/watch/#watchoptionsignored
// don't use this pattern, if you have a monorepo with linked packages.
ignored: /node_modules/,
},
});

return [
Expand Down

0 comments on commit 95aa3a3

Please sign in to comment.