Skip to content

Dispatch "pack this" to the npm PR build workflow - #173

Merged
Jake Bailey (jakebailey) merged 2 commits into
microsoft:masterfrom
jakebailey:pack-this-npm
Nov 9, 2026
Merged

Jake Bailey (jakebailey) merged 2 commits into
microsoft:masterfrom
jakebailey:pack-this-npm

Conversation

@jakebailey

Copy link
Copy Markdown
Member

PR builds are moving from the Azure DevOps tarball pipeline to a GitHub
Actions workflow in microsoft/TypeScript that publishes the build to npm
under @typescript-deploys, so that it can be installed with npm aliases
and loaded by the Playground from a CDN.
"pack this" was its only user.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

🟡 Changes recommended

The run resolver checks only candidate.name, but the new workflow places distinct_id in display_title, preventing pack this from resolving the run URL and leaving the Started link empty.

1 open issue
What changed in this PR

Moves the bot’s pack this command from Azure DevOps to the npm PR build workflow introduced in microsoft/TypeScript#65406.

Changes:

  • Dispatches pr-build.yml with the PR number and expected head commit.
  • Removes the legacy Azure build-queue helper and its request types.
File Description
GithubCommentReader/​index.ts Routes pack this through GitHub Actions and removes obsolete Azure build code.

Review effort: Balanced

Comment thread GithubCommentReader/index.ts Resolved
@jakebailey
Jake Bailey (jakebailey) merged commit 88be7e2 into microsoft:master Nov 9, 2026
5 checks passed
@jakebailey
Jake Bailey (jakebailey) deleted the pack-this-npm branch November 9, 2026 20:38
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.

3 participants