Skip to content

Sync .github/skills directory with azure-sdk-tools for PR 17350 - #50945

Merged
Helen Gao (helen229) merged 3 commits into
mainfrom
sync-.github-skills-fix/eval-mcp-app-runtime-17350
Oct 22, 2026
Merged

Helen Gao (helen229) merged 3 commits into
mainfrom
sync-.github-skills-fix/eval-mcp-app-runtime-17350

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

Sync .github/skills directory with azure-sdk-tools for PR Azure/azure-sdk-tools#17350 See eng/common workflow

Copilot AI balanced review requested due to automatic review settings October 22, 2026 21:29
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner October 22, 2026 21:29
@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 22, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
36 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 narrowly scoped evaluation updates have no unresolved findings or identified blocking regressions in static review.

No open issues 🎉

What changed in this PR

Syncs skill evaluation configurations from azure-sdk-tools PR #17350 into the Azure SDK for Java repository. These changes affect evaluation setup only, not SDK runtime code.

Changes:

  • Replaces top-level config with defaults in four evaluation suites.
  • Adds explicit skill environments and skill-invocation checks to trigger evaluations.
  • Uses next calendar month for the schedule-risk prompt to avoid a stale target date.
File Description
.github/​skills/​azsdk-common-sdk-release/​evals/​eval.yaml Updates defaults and release-skill invocation checks.
.github/​skills/​azsdk-common-sdk-breaking-change/​evals/​trigger.eval.yaml Explicitly loads the skill for trigger evaluation.
.github/​skills/​azsdk-common-prepare-release-plan/​evals/​schedule-risk.eval.yaml Makes the requested release month relative to the current date.
.github/​skills/​azsdk-common-prepare-release-plan/​evals/​eval.yaml Migrates suite settings to defaults.
.github/​skills/​azsdk-common-generate-sdk-locally/​evals/​trigger.eval.yaml Loads both relevant skills and strengthens routing expectations.
.github/​skills/​azsdk-common-generate-sdk-locally/​evals/​eval.yaml Migrates suite settings to defaults.
.github/​skills/​azsdk-common-apiview-feedback-resolution/​evals/​eval.yaml Migrates suite settings to defaults.

Review effort: Balanced

@helen229 Helen Gao (helen229) left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving the shared-skill eval sync from Azure/azure-sdk-tools#17350. All seven file changes match the source and the referenced breaking-change skill is present in this repository.

@helen229
Helen Gao (helen229) merged commit 6eeeef0 into main Oct 22, 2026
28 of 29 checks passed
@helen229
Helen Gao (helen229) deleted the sync-.github-skills-fix/eval-mcp-app-runtime-17350 branch October 22, 2026 21:46
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