Skip to content

docs: remove manual test-examples.yml edit requirement#32

Merged
leggetter merged 1 commit intomainfrom
fix/remove-test-examples-manual-edit
Feb 5, 2026
Merged

docs: remove manual test-examples.yml edit requirement#32
leggetter merged 1 commit intomainfrom
fix/remove-test-examples-manual-edit

Conversation

@leggetter
Copy link
Collaborator

Summary

Remove outdated documentation that instructed contributors to manually edit .github/workflows/test-examples.yml when adding new provider skills.

The CI workflow now uses dynamic matrix detection to automatically discover and test changed provider skills — no manual workflow edits needed.

Changes

File Update
AGENTS.md Removed from 3 checklists/process steps
CONTRIBUTING.md Removed from provider workflow and PR description template
providers.yaml Removed from header comments
scripts/validate-provider.sh Removed the grep check and error message

How Dynamic Detection Works

  • On PRs: Only skills with changes in skills/{provider}/ are tested
  • On push to main: All skills are tested

This was implemented in the detect-changes job in test-examples.yml.

Test Plan

  • ./scripts/validate-provider.sh stripe-webhooks passes
  • CI passes on this PR

Made with Cursor

The CI workflow now uses dynamic matrix detection to automatically
discover and test changed provider skills. Manual edits to
test-examples.yml are no longer needed when adding new providers.

Updated:
- AGENTS.md: Remove from checklists and step-by-step process
- CONTRIBUTING.md: Remove from provider workflow steps
- providers.yaml: Remove from header comments
- validate-provider.sh: Remove validation check

Co-authored-by: Cursor <cursoragent@cursor.com>
@leggetter leggetter merged commit 1c06e07 into main Feb 5, 2026
5 checks passed
@leggetter leggetter deleted the fix/remove-test-examples-manual-edit branch February 5, 2026 14:49
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.

1 participant