Skip to content

chore: run build-packages script on parallel #6858

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed

Conversation

uinstinct
Copy link
Contributor

@uinstinct uinstinct commented Jul 29, 2025

Description

Run build-packages script on parallel and reducing time from 12s to 3.5s.

resolves CON-3087

Checklist

  • [] I've read the contributing guide
  • [] The relevant docs, if any, have been updated or created
  • [] The relevant tests, if any, have been updated or created

Screen recording or screenshot

[ When applicable, please include a short screen recording or screenshot - this makes it much easier for us as contributors to review and understand your changes. See this PR as a good example. ]

Tests

[ What tests were added or updated to ensure the changes work as expected? ]


Summary by cubic

Replaced shell and PowerShell build scripts with a single Node.js script that builds packages in parallel, reducing build time from 12s to 3.5s.

  • Refactors
    • Removed platform-specific scripts and updated workflows to use the new cross-platform build script.

@uinstinct uinstinct requested a review from a team as a code owner July 29, 2025 12:01
@uinstinct uinstinct requested review from tomasz-stefaniak and removed request for a team July 29, 2025 12:01
@dosubot dosubot bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Jul 29, 2025
@github-project-automation github-project-automation bot moved this from Todo to In Progress in Issues and PRs Aug 4, 2025
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 4, 2025
@RomneyDa
Copy link
Collaborator

RomneyDa commented Aug 4, 2025

@uinstinct looks like several conflicts after merging #6933

@uinstinct
Copy link
Contributor Author

I think @Patrick-Erichsen integrated this in 0f582f2#diff-16273440406e457d0923a633133a3647664c5529d992ad8cb8a54939fe418391 and it is now in main!

Hence closing it.

@uinstinct uinstinct closed this Aug 5, 2025
@github-project-automation github-project-automation bot moved this from In Progress to Done in Issues and PRs Aug 5, 2025
@uinstinct uinstinct deleted the build-packages-script-js-2 branch August 5, 2025 05:16
@github-actions github-actions bot locked and limited conversation to collaborators Aug 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

3 participants