Skip to content

Publish VSIX using publish task instead of output - #7711

Merged
peterwald merged 1 commit into
dotnet:mainfrom
peterwald:pw-publish-output
Aug 24, 2026
Merged

peterwald merged 1 commit into
dotnet:mainfrom
peterwald:pw-publish-output

Conversation

@peterwald

@peterwald peterwald commented Aug 20, 2026 •

Copy link
Copy Markdown
Member

According to the 1ES docs, the correct way to publish without a PAT is to use 1ES.PublishAzureDevOpsExtension@1. The 1ES template is currently trying to use a PAT.

Microsoft Reviewers: Open in CodeFlow

Copilot AI lite review requested due to automatic review settings August 20, 2026 20:12
@peterwald
peterwald requested a review from a team as a code owner August 20, 2026 20:12

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.

Pull request overview

This PR updates the Azure DevOps pipeline’s extension publishing flow to use the dedicated 1ES publishing task (rather than publishing via a 1ES template templateContext.outputs “adoExtension” output), aligning with the stated goal of publishing without requiring a PAT.

Changes:

  • Removed the templateContext.outputs “adoExtension” publishing configuration from the publishAzureDevOpsExtension job.
  • Added an explicit 1ES.PublishAzureDevOpsExtension@1 step to publish the prepared VSIX from $(Pipeline.Workspace)\AzureDevOpsExtension.
Show a summary per file
File Description
azure-pipelines.yml Switches Azure DevOps extension publishing from a 1ES template output to the 1ES.PublishAzureDevOpsExtension@1 task using the existing service connection and VSIX path.

Review details

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

  • Files reviewed: 1/1 changed files
  • Comments generated: 0
  • Review effort level: Lite

@peterwald
peterwald enabled auto-merge (squash) August 20, 2026 21:21
@peterwald

Copy link
Copy Markdown
Member Author

@dotnet/dotnet-extensions-infra for review.

@peterwald
peterwald merged commit 2fb730e into dotnet:main Aug 24, 2026
7 checks passed
@peterwald
peterwald deleted the pw-publish-output branch August 24, 2026 16:56
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 24, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants