Add Claude plugin manifests for all plugins - #1071
Conversation
Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
|
⏭️ No skills to evaluate — no changed skills with tests were found in this PR. View workflow run |
There was a problem hiding this comment.
Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.
Note
This error may be related to your runner configuration. You can now configure runners for Copilot code review separately from Copilot cloud agent by creating a copilot-code-review.yml file with your setup steps. Read the docs for details.
|
✅ Evaluation passed for |
|
You beat me to it! Thanks for fixing. I'm hoping we'd be able to simplify some of this once we move over to Agent Plugins 1.0 #1006 . |
|
✅ Approved by @AbhitejJohn. cc @dotnet/skills-merge-approvers — ready to merge. |
There was a problem hiding this comment.
Review details
Suppressed comments (1)
.github/workflows/plugin-versioning-tests.yml:18
- Same as above for the
pushtrigger: add.codex-plugin/plugin.jsonso main-branch Codex manifest-only updates still execute the versioning regression tests.
- "eng/version/**"
- "plugins/*/plugin.json"
- "plugins/*/.claude-plugin/plugin.json"
- Files reviewed: 21/21 changed files
- Comments generated: 1
- Review effort level: Lite
| - "eng/version/**" | ||
| - "plugins/*/plugin.json" | ||
| - "plugins/*/.claude-plugin/plugin.json" |
Summary
.claude-plugin/plugin.jsonto all 15 plugins that were missing itFixes #1029