Skip to content

Sync .github/skills directory with azure-sdk-tools for PR 16634 - #40085

Merged
Crystal YU (chunyu3) merged 28 commits into
mainfrom
sync-.github-skills-yuc/sdk-quality-skill-16634
Sep 24, 2026
Merged

Crystal YU (chunyu3) merged 28 commits into
mainfrom
sync-.github-skills-yuc/sdk-quality-skill-16634

Conversation

@azure-sdk-automation

Copy link
Copy Markdown
Contributor

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

Crystal YU (chunyu3) and others added 28 commits September 24, 2026 17:54
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: chunyu3 <10458143+chunyu3@users.noreply.github.com>
Copilot AI balanced review requested due to automatic review settings September 24, 2026 18:05
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner September 24, 2026 18:05
@azure-sdk-automation
azure-sdk-automation Bot requested a review from a team as a code owner September 24, 2026 18:05
@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 Sep 24, 2026
@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines:
9 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.

Copilot review overview

🟡 Changes recommended

The workflow contains conflicting instructions, references a missing skill, and adds an eval that cannot satisfy its mandatory checkpoint flow.

Get a fresh assessment by requesting another Copilot review.

Review effort: Balanced
Findings: 2 Medium severity

Open (2)
What changed in this PR

Syncs shared Azure SDK skills and evals for breaking-change detection and local SDK generation.

Changes:

  • Adds a breaking-change detection/mitigation skill and fixture.
  • Expands local generation behavior and eval coverage.
  • Adds skill-trigger evaluations.
File Description
.github/​skills/​azsdk-common-sdk-breaking-change/​SKILL.md Defines breaking-change workflow.
.../​fixtures/​.../​models.tsp Provides Web PubSub evaluation fixture.
.../​evals/​trigger.eval.yaml Tests skill triggering.
.../​evals/​eval.yaml Tests breaking-change detection.
.github/​skills/​azsdk-common-generate-sdk-locally/​SKILL.md Refines generation modes and routing.
.../​evals/​trigger.eval.yaml Adds generation trigger tests.
.../​evals/​eval.yaml Expands full and generate-only evaluations.

Comment thread .github/skills/azsdk-common-generate-sdk-locally/SKILL.md
Comment thread .github/skills/azsdk-common-generate-sdk-locally/evals/eval.yaml

Copilot AI commented Sep 24, 2026

Copy link
Copy Markdown
Contributor

One or more custom setup steps configured for this repository failed during this Copilot code review run:

Install library dependencies

Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review.

Note

You can configure setup steps for Copilot code review separately from Copilot cloud agent with a copilot-code-review.yml file. Read the docs for details.

@chunyu3
Crystal YU (chunyu3) merged commit a51c5fb into main Sep 24, 2026
125 of 127 checks passed
@chunyu3
Crystal YU (chunyu3) deleted the sync-.github-skills-yuc/sdk-quality-skill-16634 branch September 24, 2026 23:20
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