Skip to content

Remove unused Publish-Build-Assets variable group from components-ai-daily.yml - #68635

Merged
wtgodbe merged 1 commit into
mainfrom
akoeplinger-patch-1
Aug 24, 2026
Merged

Remove unused Publish-Build-Assets variable group from components-ai-daily.yml#68635
wtgodbe merged 1 commit into
mainfrom
akoeplinger-patch-1

Conversation

@akoeplinger

Copy link
Copy Markdown
Member

No description provided.

Copilot AI lite review requested due to automatic review settings August 19, 2026 08:42
@akoeplinger
akoeplinger requested review from a team and wtgodbe as code owners August 19, 2026 08:42
@akoeplinger
akoeplinger requested a review from ilonatommy August 19, 2026 08:43
@akoeplinger
akoeplinger enabled auto-merge (squash) August 19, 2026 08:43

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

Removes an unused Azure DevOps variable group reference from the Components.AI daily pipeline, reducing reliance on unnecessary secret/variable injections while keeping the publish flow unchanged.

Changes:

  • Removed the Publish-Build-Assets variable group from .azure/pipelines/components-ai-daily.yml.
  • Kept asset publishing via the existing publish-build-assets.yml template (which uses the Darc: Maestro Production service connection and System.AccessToken, not the removed variables).

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

@wtgodbe
wtgodbe disabled auto-merge August 24, 2026 17:20
@wtgodbe
wtgodbe merged commit 1cee9a7 into main Aug 24, 2026
28 checks passed
@wtgodbe
wtgodbe deleted the akoeplinger-patch-1 branch August 24, 2026 17:20
@dotnet-milestone-bot dotnet-milestone-bot Bot added this to the 12.0-preview1 milestone Aug 25, 2026
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