Skip to content

Don't validate extension during publish step - #7725

Merged
peterwald merged 2 commits into
dotnet:mainfrom
peterwald:pw-validate-vsix
Aug 31, 2026
Merged

peterwald merged 2 commits into
dotnet:mainfrom
peterwald:pw-validate-vsix

Conversation

@peterwald

@peterwald peterwald commented Aug 31, 2026 •

Copy link
Copy Markdown
Member
Microsoft Reviewers: Open in CodeFlow

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

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

Review tier: Lite
Findings: 1 Low severity

New issues introduced by this change (1)
Severity Finding
Low severity azure-pipelines.yml — This line has trailing whitespace after false, which can create unnecessary diffs and may failâ€Ķ
What changed in this PR

Disables Azure DevOps extension validation during the publish step in the main Azure Pipelines definition, aligning with the intent to avoid validation at publish time.

Changes:

  • Set validateExtension to false for the 1ES.PublishAzureDevOpsExtension@1 “Publish Extension” task.
File Description
azure-pipelines.yml Disables extension validation during the publish step for the VSIX.

ðŸ’Ą Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.

Comment thread azure-pipelines.yml Outdated
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
@peterwald
peterwald enabled auto-merge (squash) August 31, 2026 17:49
@peterwald
peterwald merged commit 258da35 into dotnet:main Aug 31, 2026
5 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Oct 1, 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