Skip to content

Sync eng/common directory with azure-sdk-tools for PR 16449 - #10218

Merged
Travis Prescott (tjprescott) merged 2 commits into
mainfrom
sync-eng-common-arh/InitialPipelines-16449
Aug 7, 2026
Merged

Sync eng/common directory with azure-sdk-tools for PR 16449#10218
Travis Prescott (tjprescott) merged 2 commits into
mainfrom
sync-eng-common-arh/InitialPipelines-16449

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

Copilot AI balanced review requested due to automatic review settings August 7, 2026 16:57
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner August 7, 2026 16:57
@azure-sdk-automation azure-sdk-automation Bot added EngSys This issue is impacting the engineering system. Central-EngSys This issue is owned by the Engineering System team. labels Aug 7, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
1 pipeline(s) were filtered out due to trigger conditions.
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.

Pull request overview

Note

Copilot could not run the full agentic suite for this review because it was automatically requested on a bot-authored pull request. Request a review from Copilot under Reviewers to retry with the full agentic suite. Improved support for bot-authored pull requests is coming soon.

Introduces reusable Azure Pipelines YAML templates to standardize API Review Hub artifact generation and repository preparation across languages.

Changes:

  • Added a base step template to prepare directories, checkout a validated Git ref, and validate/write artifact metadata.
  • Added a base job template to clone the source repository and provide common variables/hooks for language-specific setup and generation steps.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.

File Description
eng/common/pipelines/templates/steps/create-apireview-hub-artifacts-base.yml Adds shared steps for checkout + artifact contract validation/metadata emission.
eng/common/pipelines/templates/jobs/apireview-hub-job-base.yml Adds a shared job wrapper to clone the source repo and compose setup/generation steps.

$stateDir = '${{ parameters.workingDir }}/state'
$apiMdPath = Join-Path $outputDir 'api.md'
$apiMetadataPath = Join-Path $outputDir 'api.metadata.yml'
$packageRelativePathPath = Join-Path $stateDir 'package-relative-path.txt'
$ErrorActionPreference = 'Stop'

$outputDir = '${{ parameters.outputDir }}'
$stateDir = '${{ parameters.workingDir }}/state'
source_repo="${{ parameters.sourceCheckoutDir }}"
repository_full_name="${{ parameters.sourceRepositoryFullName }}"

rm -rf "$source_repo"
mkdir -p "$(dirname "$source_repo")"

echo "Cloning $repository_full_name into $source_repo"
git clone --filter=blob:none --no-checkout "https://github.com/$repository_full_name.git" "$source_repo"
@tjprescott
Travis Prescott (tjprescott) merged commit 8a529fe into main Aug 7, 2026
8 of 9 checks passed
@tjprescott
Travis Prescott (tjprescott) deleted the sync-eng-common-arh/InitialPipelines-16449 branch August 7, 2026 17:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Central-EngSys This issue is owned by the Engineering System team. EngSys This issue is impacting the engineering system.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants