Skip to content

docs: add eval requirements to contribution and agent guidelines#160

Merged
marc0olo merged 1 commit intomainfrom
docs/eval-requirements
Apr 14, 2026
Merged

docs: add eval requirements to contribution and agent guidelines#160
marc0olo merged 1 commit intomainfrom
docs/eval-requirements

Conversation

@marc0olo
Copy link
Copy Markdown
Member

Summary

  • CLAUDE.md: remind agents to consider eval coverage when adding new behaviors or pitfalls to a skill; add PR eval requirements (new skill vs. improvement, <details> collapse format)
  • CONTRIBUTING.md: make eval results required (not optional) for new skill PRs; clarify that skill improvements only need results for new eval cases; add <details> collapse format example

These changes ensure both human contributors and agents working in this repo know the eval expectations before submitting a PR.

- CLAUDE.md: remind agents to consider evals when modifying skills,
  add PR eval requirements (new skill vs improvement, details block)
- CONTRIBUTING.md: make eval results required for new skills in PRs,
  clarify that improvements only need results for new eval cases,
  add <details> collapse format example
@marc0olo marc0olo requested review from a team and JoshDFN as code owners April 10, 2026 08:46
@github-actions
Copy link
Copy Markdown

Skill Validation Report

No skill files were changed in this PR — validation skipped.

@marc0olo marc0olo merged commit 1d125a9 into main Apr 14, 2026
6 checks passed
@marc0olo marc0olo deleted the docs/eval-requirements branch April 14, 2026 16:30
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.

2 participants