Skip to content

Conversation

@puehringer
Copy link
Contributor

Developer Checklist (Definition of Done)

Issue

  • All acceptance criteria from the issue are met
  • Tested in latest Chrome/Firefox

UI/UX/Vis

  • Requires UI/UX/Vis review
    • Reviewer(s) are notified (tag assignees)
    • Review has occurred (link to notes)
    • Feedback is included in this PR
    • Reviewer(s) approve of concept and design

Code

  • Branch is up-to-date with the branch to be merged with, i.e., develop
  • Code is cleaned up and formatted
  • Unit tests are written (frontend/backend if applicable)
  • Integration tests are written (if applicable)

PR

  • Descriptive title for this pull request is provided (will be used for release notes later)
  • Reviewer and assignees are defined
  • Add type label (e.g., bug, feature) to this pull request
  • Add release label (e.g., release: minor) to this PR following semver
  • The PR is connected to the corresponding issue (via Closes #...)
  • Summary of changes is written

Summary of changes

  • Increases main heap (max-old-space-size) and "new" heap (max-semi-space-size) to improve performance of all node scripts, i.e. ESLint:
Before:
TIME=10 yarn run lint  161,44s user 4,93s system 168% cpu 1:38,77 total
After:
TIME=10 yarn run lint  98,23s user 4,23s system 160% cpu 1:03,76 total

Screenshots

Additional notes for the reviewer(s)

Thanks for creating this pull request 🤗

@puehringer puehringer requested a review from a team as a code owner November 1, 2025 09:17
@puehringer puehringer added release: patch PR merge results in a new patch version type: feature New feature or request labels Nov 1, 2025
@puehringer puehringer merged commit 15a7218 into develop Nov 1, 2025
1 check passed
@puehringer puehringer deleted the mp/node_space_size branch November 1, 2025 09:25
@github-actions github-actions bot mentioned this pull request Nov 1, 2025
puehringer added a commit that referenced this pull request Nov 1, 2025
* chore: prepare next dev release

* feat: set max space size for all node scripts (#171)

* Prepare release version 15.1.2

---------

Co-authored-by: datavisyn-bot <>
Co-authored-by: Michael Pühringer <51900829+puehringer@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

release: patch PR merge results in a new patch version type: feature New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant