Skip to content

Disable publish template steps for VSIX publish jobs - #64559

Merged
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
jakebailey:vscode-publish-authorization
Sep 30, 2026
Merged

Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
jakebailey:vscode-publish-authorization

Conversation

@jakebailey

Copy link
Copy Markdown
Member

If these are enabled, the template injects ESRP signing setup steps, which we don't use.

Copilot AI balanced review requested due to automatic review settings September 30, 2026 17:32
@typescript-automation typescript-automation Bot added Author: Team For Uncommitted Bug PR for untriaged, rejected, closed or missing bug labels Sep 30, 2026

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

🟢 Approval recommended

The configuration matches the established opt-out pattern and correctly targets both affected jobs.

Review effort: Balanced
Findings: None

What changed in this PR

Disables unused ESRP signing setup in the VSIX validation and publication jobs.

Changes:

  • Sets templateContext.mb.publish.enabled to false for both jobs.
File Description
tools/​pipelines/​vscode-typescript-publish.yml Disables template-injected publish steps.

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

@jakebailey
Jake Bailey (jakebailey) added this pull request to the merge queue Sep 30, 2026
Merged via the queue into microsoft:main with commit 79feed2 Sep 30, 2026
29 checks passed
@jakebailey
Jake Bailey (jakebailey) deleted the vscode-publish-authorization branch September 30, 2026 18:09
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

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants