Skip to content

Build all platforms in parallel in release pipeline - #64563

Draft
Jake Bailey (jakebailey) wants to merge 4 commits into
microsoft:mainfrom
jakebailey:parallel-release-build
Draft

Jake Bailey (jakebailey) wants to merge 4 commits into
microsoft:mainfrom
jakebailey:parallel-release-build

Conversation

@jakebailey

Copy link
Copy Markdown
Member

This should speed things up, especially full release builds with lots of platforms.

Matrix setup and platform builds do not sign artifacts, so avoid provisioning signing prerequisites in those jobs.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟡 Changes recommended

Undefined helpers, lost executable permissions, and profile-specific artifact paths currently break supported build flows.

Review effort: Balanced
Findings: 4 High severity

Open (4)
What changed in this PR

Splits release package builds across parallel platform jobs, then assembles them for signing and packaging.

Changes:

  • Adds a dynamic platform build matrix.
  • Separates main and platform package tasks.
  • Downloads and assembles platform artifacts before testing and signing.
File Description
Herebyfile.mjs Adds granular package-build tasks and matrix generation.
tools/​pipelines/​typescript-build.yml Runs platform builds in parallel and assembles their artifacts.

💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread Herebyfile.mjs Outdated
Comment thread Herebyfile.mjs Outdated
Comment thread tools/pipelines/typescript-build.yml Outdated
Comment thread tools/pipelines/typescript-build.yml
Rebased build tasks must use the current process helpers, and platform artifacts must retain their profile-specific paths and executable permissions after transfer.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🔵 Needs a closer look

Release correctness depends on Azure matrix expansion and artifact transfer behavior that requires pipeline-level validation.

Review effort: Balanced
Findings: None

Resolved since last review (4)

Azure appends the matrix leg name to job titles but leaves platform macros unexpanded, so give each leg a readable static title.

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants