Skip to content

Switch to the arcade-skills plugin for ci-analysis#127284

Merged
agocke merged 1 commit intodotnet:mainfrom
lewing:switch-to-arcade-skills
Apr 22, 2026
Merged

Switch to the arcade-skills plugin for ci-analysis#127284
agocke merged 1 commit intodotnet:mainfrom
lewing:switch-to-arcade-skills

Conversation

@lewing
Copy link
Copy Markdown
Member

@lewing lewing commented Apr 22, 2026

Also remove the very stale vmr-codeflow-status which is replaced by complementary skills in the https://github.com/dotnet/arcade-skills repo

Copilot AI review requested due to automatic review settings April 22, 2026 15:58
@lewing lewing added the area-skills Agent Skills label Apr 22, 2026
@lewing lewing requested a review from agocke April 22, 2026 15:58
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR removes the in-repo implementations of the ci-analysis and vmr-codeflow-status skills and adds configuration to source skills from the dotnet/arcade-skills marketplace plugin instead.

Changes:

  • Removed local PowerShell scripts and reference docs for ci-analysis and vmr-codeflow-status.
  • Added Copilot and Claude settings to enable the dotnet-dnceng@dotnet-arcade-skills plugin from dotnet/arcade-skills.

Reviewed changes

Copilot reviewed 16 out of 16 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.github/skills/vmr-codeflow-status/scripts/Get-CodeflowStatus.ps1 Removed local VMR codeflow analysis script (expected to be replaced by plugin).
.github/skills/vmr-codeflow-status/references/vmr-codeflow-reference.md Removed local reference documentation for VMR codeflow.
.github/skills/vmr-codeflow-status/references/vmr-build-topology.md Removed local reference documentation for VMR build topology.
.github/skills/vmr-codeflow-status/SKILL.md Removed local skill definition for vmr-codeflow-status.
.github/skills/ci-analysis/scripts/Get-CIStatus.ps1 Removed local CI analysis script (expected to be replaced by plugin).
.github/skills/ci-analysis/references/sql-tracking.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/manual-investigation.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/helix-artifacts.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/delegation-patterns.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/build-progression-analysis.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/binlog-comparison.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/azure-cli.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/references/azdo-helix-reference.md Removed local CI analysis reference doc.
.github/skills/ci-analysis/SKILL.md Removed local skill definition for ci-analysis.
.github/copilot/settings.json Enables dotnet-dnceng@dotnet-arcade-skills plugin and registers dotnet/arcade-skills marketplace source for Copilot.
.claude/settings.json Mirrors the same plugin + marketplace configuration for Claude.

Comment thread .github/copilot/settings.json
Comment thread .github/copilot/settings.json
@lewing lewing requested a review from kotlarmilos April 22, 2026 16:45
@agocke agocke enabled auto-merge (squash) April 22, 2026 22:48
@agocke agocke disabled auto-merge April 22, 2026 22:48
@agocke
Copy link
Copy Markdown
Member

agocke commented Apr 22, 2026

/ba-g docs-only change

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-skills Agent Skills

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants