Skip to content

feat: solve the problem of memory overflow in packaging#112

Merged
hexqi merged 3 commits into
devfrom
wyp/space-size-0402
Apr 3, 2026
Merged

feat: solve the problem of memory overflow in packaging#112
hexqi merged 3 commits into
devfrom
wyp/space-size-0402

Conversation

@wuyiping0628
Copy link
Copy Markdown
Contributor

@wuyiping0628 wuyiping0628 commented Apr 2, 2026

Summary by CodeRabbit

  • Chores
    • Optimized build process performance across deployment workflows.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 2, 2026

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Free

Run ID: 2a973ee4-6fad-4582-90c3-ac042b242250

📥 Commits

Reviewing files that changed from the base of the PR and between 1e9469a and a6e8a9d.

📒 Files selected for processing (3)
  • .github/workflows/build-home.yml
  • .github/workflows/deploy-github.yml
  • .github/workflows/deploy-obs.yml
✅ Files skipped from review due to trivial changes (1)
  • .github/workflows/deploy-github.yml
🚧 Files skipped from review as they are similar to previous changes (2)
  • .github/workflows/build-home.yml
  • .github/workflows/deploy-obs.yml

Walkthrough

Three GitHub Actions workflows now set NODE_OPTIONS="--max-old-space-size=8192" for the "Build home" step, adjusting the Node.js heap size used by pnpm build.

Changes

Cohort / File(s) Summary
GitHub Actions — Build home step
.​github/workflows/build-home.yml, .​github/workflows/deploy-github.yml, .​github/workflows/deploy-obs.yml
Added NODE_OPTIONS: --max-old-space-size=8192 to the environment for the "Build home" step so pnpm build runs with increased Node.js max old space size. No other steps or build commands changed.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Poem

🐰 I nibbled logs and found the trace,

Eight gigs now warm my build-time space.
No more crumbs of memory flown,
The pipeline hums — a garden grown.
Hooray, said Rabbit, with a happy pace.


Note

🎁 Summarized by CodeRabbit Free

Your organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login.

Comment @coderabbitai help to get the list of available commands and usage tips.

Comment thread .github/workflows/build-home.yml Outdated
@hexqi hexqi merged commit 5ef7250 into dev Apr 3, 2026
1 of 2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants