Uh oh!
There was an error while loading. Please reload this page.
Remove Publish-Build-Assets variable group from main - #188
Merged
akoeplinger merged 1 commit intoAug 19, 2026
Merged
Conversation
Synchronize the relevant Arcade template cleanup for VG20 retirement. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: e890b71a-c1aa-416c-a15c-be8da9fdd9b4
Contributor
There was a problem hiding this comment.
Pull request overview
Removes the Publish-Build-Assets Azure DevOps variable group from shared Arcade pipeline templates and stops passing a couple of now-obsolete secret variables into the log redaction step, aligning this repo’s main branch with the upstream credential-removal work.
Changes:
- Removed
Publish-Build-Assetsvariable group imports from shared variable/job templates. - Updated the
publish-logs.ymlredaction invocation to no longer include two obsolete secret values.
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| eng/common/core-templates/steps/publish-logs.yml | Removes two obsolete secret variables from the explicit redaction argument list. |
| eng/common/core-templates/post-build/common-variables.yml | Stops importing the Publish-Build-Assets variable group at the common post-build variable layer. |
| eng/common/core-templates/jobs/codeql-build.yml | Removes the Publish-Build-Assets variable group from the CodeQL build job template. |
| eng/common/core-templates/job/publish-build-assets.yml | Removes the Publish-Build-Assets variable group from the asset publishing job’s internal-only variable setup. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
akoeplinger
approved these changes
Aug 19, 2026
Uh oh!
There was an error while loading. Please reload this page.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Synchronizes the relevant Arcade credential-removal changes on
mainso the pipeline no longer imports VG20 or passes its obsolete log-redaction values.Tracked by https://dev.azure.com/dnceng/internal/_workitems/edit/12131