Skip to content

Sync eng/common directory with azure-sdk-tools for PR 17316 - #3996

Merged
Travis Prescott (tjprescott) merged 2 commits into
mainfrom
sync-eng-common-arh/pipelineTweaks-17316
Oct 19, 2026
Merged

Travis Prescott (tjprescott) merged 2 commits into
mainfrom
sync-eng-common-arh/pipelineTweaks-17316

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync eng/common directory with azure-sdk-tools for PR Azure/azure-sdk-tools#17316 See eng/common workflow

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
There may be pipelines that require an authorized user to comment /azp run to run.

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.

🟢 Approval recommended

The focused template changes consistently propagate pinned commits, with no blocking issues identified; pipelines were not executed.

No open issues 🎉

What changed in this PR

Syncs shared engineering templates with azure-sdk-tools PR #17316 to pin API review artifact generation to specific commits.

Changes:

  • Validates, fetches, and verifies the requested commit SHA.
  • Includes the requested source commit in failure callbacks.
File Description
eng/​common/​pipelines/​templates/​steps/​apireview-hub-failure-callback.yml Adds source commit information to failure callbacks.
eng/​common/​pipelines/​templates/​jobs/​apireview-hub-generate-artifact-base.yml Checks out and verifies pinned base or target commits.

Review effort: Balanced

@tjprescott
Travis Prescott (tjprescott) merged commit 2d31822 into main Oct 19, 2026
15 of 26 checks passed
@tjprescott
Travis Prescott (tjprescott) deleted the sync-eng-common-arh/pipelineTweaks-17316 branch October 19, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants