Skip to content

Commit

Permalink
fix(ci): NODE_OPTIONS: --max-old-space-size=4096
Browse files Browse the repository at this point in the history
to fix node builds for now.

Rollup is known to require quite a bit of memory
when bundling with source maps enabled.
https://rollupjs.org/troubleshooting/#error-javascript-heap-out-of-memory

Signed-off-by: Max <max@nextcloud.com>
  • Loading branch information
max-nextcloud committed May 13, 2024
1 parent f765dd7 commit cac240c
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 0 deletions.
2 changes: 2 additions & 0 deletions .github/workflows/cypress-e2e.yml
Original file line number Diff line number Diff line change
Expand Up @@ -67,6 +67,8 @@ jobs:

- name: Install node dependencies & build app
working-directory: apps/${{ env.APP_NAME }}
env:
NODE_OPTIONS: --max-old-space-size=4096
run: |
npm ci
TESTING=true npm run build --if-present
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/node-test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -69,6 +69,7 @@ jobs:
- name: Install dependencies & build
env:
CYPRESS_INSTALL_BINARY: 0
NODE_OPTIONS: --max-old-space-size=4096
run: |
npm ci
npm run build --if-present
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/node.yml
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,7 @@ jobs:
env:
CYPRESS_INSTALL_BINARY: 0
PUPPETEER_SKIP_DOWNLOAD: true
NODE_OPTIONS: --max-old-space-size=4096
run: |
npm ci
npm run build --if-present
Expand Down

0 comments on commit cac240c

Please sign in to comment.