Skip to content

Conversation

@patricijabrecko
Copy link
Contributor

Description

This pull request sets a memory limit for local Node.js processes to 700MB by adding the configuration option to the .npmrc file.

Connected Issues

https://app.devrev.ai/devrev/works/ISS-141117

Checklist

  • Tests added/updated and ran with npm run test OR no tests needed.
  • Code formatted and checked with npm run lint.
  • Added "How to test" section to the description OR this section is not needed.

Copilot AI review requested due to automatic review settings September 22, 2025 10:56
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request adds a memory limit configuration for local Node.js processes by setting a 700MB memory limit through the .npmrc file.

  • Adds node-options=--max-old-space-size=700 to .npmrc to limit Node.js memory usage

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

Copy link
Contributor

@gasperzgonec gasperzgonec left a comment

Choose a reason for hiding this comment

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

LGTM.

Copy link
Contributor

@radovanjorgic radovanjorgic left a comment

Choose a reason for hiding this comment

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

LG

@patricijabrecko patricijabrecko merged commit d2a9246 into main Sep 22, 2025
2 checks passed
@patricijabrecko patricijabrecko deleted the node-memory-limit branch September 22, 2025 11:26
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