Skip to content

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

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

Praveen Kuttappan (praveenkuttappan) 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:
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.

🟡 Changes recommended

Required invocation checks reference a missing skill definition, and the new anti-trigger conflicts with the generation skill’s documented routing.

2 open issues
What changed in this PR

Syncs Azure SDK skill evaluations from azure-sdk-tools PR #17350, updating evaluation configuration and skill-routing checks.

Changes:

  • Replaces top-level config with defaults in four evaluation suites.
  • Adds generation and breaking-change trigger tests and an explicit release skill-invocation check.
  • Uses next calendar month for the schedule-risk test’s requested release target.
File Description
.github/​skills/​azsdk-common-sdk-release/​evals/​eval.yaml Updates defaults and release skill-invocation validation.
.github/​skills/​azsdk-common-sdk-breaking-change/​evals/​trigger.eval.yaml Adds a breaking-change skill trigger evaluation.
.github/​skills/​azsdk-common-prepare-release-plan/​evals/​schedule-risk.eval.yaml Makes the requested release target relative to the current date.
.github/​skills/​azsdk-common-prepare-release-plan/​evals/​eval.yaml Migrates evaluation settings to defaults.
.github/​skills/​azsdk-common-generate-sdk-locally/​evals/​trigger.eval.yaml Adds generation and breaking-change routing checks.
.github/​skills/​azsdk-common-generate-sdk-locally/​evals/​eval.yaml Migrates evaluation settings to defaults.
.github/​skills/​azsdk-common-apiview-feedback-resolution/​evals/​eval.yaml Migrates evaluation settings to defaults.

Review effort: Balanced

@praveenkuttappan
Praveen Kuttappan (praveenkuttappan) merged commit 757a2f5 into main Oct 22, 2026
16 of 18 checks passed
@praveenkuttappan
Praveen Kuttappan (praveenkuttappan) deleted the sync-.github-skills-fix/eval-mcp-app-runtime-17350 branch October 22, 2026 21:57
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.

4 participants