Skip to content

Sync eng/common directory with azure-sdk-tools for PR 17367 - #50984

Open
azure-sdk-automation[bot] wants to merge 1 commit into
mainfrom
sync-eng-common-samvaity/ccr-review-quality-evals-17367
Open

azure-sdk-automation[bot] wants to merge 1 commit into
mainfrom
sync-eng-common-samvaity/ccr-review-quality-evals-17367

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

@azure-sdk-automation
azure-sdk-automation Bot requested review from a team as code owners October 26, 2026 21:22
@azure-sdk-automation
azure-sdk-automation Bot requested review from Sameeksha Vaity (samvaity) and a balanced review from Copilot October 26, 2026 21:22
@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 Oct 26, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
Successfully started running 1 pipeline(s).
35 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.

🟢 Approval recommended

The narrow, additive discovery changes preserve existing behavior, with no identified regressions.

No open issues 🎉

What changed in this PR

Syncs the shared skill-evaluation pipeline with Azure SDK Tools PR #17367 to include two additional evaluation files.

Changes:

  • Adds explicit discovery entries for TypeSpec naming and code-review evaluations.
  • Preserves the existing evaluation glob and shard timeout.
File Description
eng/​common/​pipelines/​skill-eval.yml Adds two eval.yaml paths to evaluation discovery.

Review effort: Balanced

This branch has not been deployed

No deployments
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.

3 participants