Skip to content

Revert sync from azure-sdk-tools PR 16412 - #48247

Merged
praveenkuttappan merged 1 commit into
mainfrom
revert/sync-pr-16412
Jul 24, 2026
Merged

Revert sync from azure-sdk-tools PR 16412#48247
praveenkuttappan merged 1 commit into
mainfrom
revert/sync-pr-16412

Conversation

@helen229

Copy link
Copy Markdown
Member

Reverts the #16412 sync changes merged into this repository.

Reverts:

Copilot AI review requested due to automatic review settings July 24, 2026 17:49
@helen229
helen229 requested review from a team, benbp and l0lawrence as code owners July 24, 2026 17:49
@azure-pipelines

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

Pull request overview

Reverts the eval-authoring content introduced by sync PRs #48219 and #48220.

Changes:

  • Removes the shared eval-authoring guide and three related skills/evals.
  • Restores the earlier skill-eval glob configuration.

Reviewed changes

Copilot reviewed 8 out of 8 changed files in this pull request and generated 1 comment.

Show a summary per file
File Description
eng/common/pipelines/skill-eval.yml Reverts plain eval.yaml discovery.
eng/common/knowledge/eval-authoring/README.md Removes the shared authoring guide.
.github/skills/azsdk-common-eval-authoring-workflow/SKILL.md Removes the workflow authoring skill.
.github/skills/azsdk-common-eval-authoring-workflow/evals/eval.yaml Removes its routing evals.
.github/skills/azsdk-common-eval-authoring-tool/SKILL.md Removes the tool authoring skill.
.github/skills/azsdk-common-eval-authoring-tool/evals/eval.yaml Removes its routing evals.
.github/skills/azsdk-common-eval-authoring-skill/SKILL.md Removes the skill-eval authoring skill.
.github/skills/azsdk-common-eval-authoring-skill/evals/eval.yaml Removes its routing evals.

# Second entry catches the plain `eval.yaml` name (merged routing+capability file);
# it doesn't match *.eval.yaml since "eval.yaml" itself is shorter than that suffix.
evalGlobs:
- '*/evals/*.eval.yaml'
@github-actions

Copy link
Copy Markdown
Contributor
[Pilot] PR Pipeline Failure Analysis

A CI pipeline failed on this pull request. Here is an automated analysis of what went wrong and how to get the build green.

What failed

The sync-.github-skills pipeline validation failed because this PR modifies files under the eng/common/ directory (eng/common/knowledge/eval-authoring/README.md and eng/common/pipelines/skill-eval.yml). Changes to eng/common are not allowed in this repository — they must be made in the upstream azure-sdk-tools repo and then synced in.

Build #6614359

Recommended next steps

  • Remove the changes to eng/common/knowledge/eval-authoring/README.md and eng/common/pipelines/skill-eval.yml from this PR.
  • Submit those changes to Azure/azure-sdk-tools following the common engineering system workflow.
  • Once the changes are synced to this repo via the normal eng/common sync process, they will appear here automatically.
  • See the CI troubleshooting guide: https://aka.ms/ci-fix
  • Push new commits to address the failures; this comment updates automatically on the next failing run.
Raw pipeline analysis (azsdk ci analyze)
Failed Tasks
--------------------------------------------------------------------------------
### Errors:
Changes to files under 'eng/common' directory should not be made in this Repo
eng/common/knowledge/eval-authoring/README.md eng/common/pipelines/skill-eval.yml
Follow the workflow at https://github.com/Azure/azure-sdk-tools/blob/main/doc/common/common_engsys.md

### Pipeline: https://dev.azure.com/azure-sdk/public/_build/results?buildId=6614359

Copilot detected the failing pipeline and generated the analysis above. To have it attempt a fix automatically, reply with @copilot please fix the failing pipeline on this PR.

Generated by Pipeline Analysis - Next Steps · 18.1 AIC · ⌖ 6.16 AIC · ⊞ 6.7K ·

@praveenkuttappan
praveenkuttappan merged commit fdcaeaf into main Jul 24, 2026
21 of 23 checks passed
@praveenkuttappan
praveenkuttappan deleted the revert/sync-pr-16412 branch July 24, 2026 18:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants