Skip to content

Conversation

@pilgd-aws
Copy link

@pilgd-aws pilgd-aws commented Jul 9, 2025

Checklist

Unsure which checkboxes above I need for this change

PR Summary

  • Update version from 1.6.1 to 1.5.1
  • Add memory optimizations to prevent build failures:
    • Limit Node.js heap size to 2GB (Node ate too much memory in previous PRs even with no changes, ex: readme change test #83)
    • Reduce thread pool size to 2
    • Disable webpack parallelization
    • Limit yarn network concurrency
  • Tested successfully in CodeBuild environment as well

Fixes build failures caused by memory exhaustion during webpack bundling step.

@conda-forge-admin
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/meta.yaml) and found it was in an excellent condition.

@pilgd-aws pilgd-aws changed the base branch from main to 1.5 July 9, 2025 22:52
@aws-prayags
Copy link
Contributor

@conda-forge-admin please rerender

@goel-akas goel-akas merged commit 3530b32 into conda-forge:1.5 Jul 10, 2025
4 checks passed

export DISABLE_V8_COMPILE_CACHE=1

# Limit Node.js memory usage to prevent OOM kills
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

would be good to link a failed run here for context

jq 'del(.dependencies."@vscode/ripgrep")' package.json.orig > package.json

yarn install
yarn install --network-concurrency 1
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

we should add comment noting how this helps

Comment on lines +18 to +19
- NODE_OPTIONS=--max-old-space-size=4096
- UV_THREADPOOL_SIZE=4
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

to check if this has to be specified both in meta.yaml and build.sh

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.

4 participants