Skip to content

Conversation

@JanProvaznik
Copy link
Contributor

s360 item

@JanProvaznik JanProvaznik enabled auto-merge (squash) January 5, 2026 09:49
Copy link

Copilot AI left a comment

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 migrates the release pipeline to use the 1ES governed pipeline template, aligning with Microsoft's security and compliance requirements for official build pipelines.

Key changes:

  • Adopts the 1ES Official Pipeline Template (v1) with the extends pattern
  • Adds network isolation policy configuration set to "Permissive,CFSClean"
  • Restructures the pipeline to nest all stages under the template extension while preserving existing job logic

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

- download: microsoft_MSBuildLocator
displayName: 'Download Symbols Artifact'
artifact: Symbols
name: VSEngSS-MicroBuild2022-1ES
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

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

Trailing whitespace detected at the end of this line. This should be removed to maintain consistent formatting.

Suggested change
name: VSEngSS-MicroBuild2022-1ES
name: VSEngSS-MicroBuild2022-1ES

Copilot uses AI. Check for mistakes.
Comment on lines +63 to +65
- job: PushToNuGet
displayName: 'Push to nuget.org'
steps:
Copy link

Copilot AI Jan 5, 2026

Choose a reason for hiding this comment

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

The pool specification has been removed from the PushToNuGet job. While a default pool is configured at the template level (line 26-27), verify that this job should use the default pool rather than having an explicit pool specification. The RetainBuildJob and PublicNuGetRelease jobs previously used the same pool, but PushToNuGet had its own pool definition that is now removed.

Copilot uses AI. Check for mistakes.
@JanProvaznik JanProvaznik merged commit c6d13f2 into microsoft:main Jan 5, 2026
8 checks passed
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.

2 participants