Skip to content

docs: add reviewer guidance and author attestation to PR template#4106

Merged
maxrjones merged 3 commits into
zarr-developers:mainfrom
maxrjones:chore/pr-template
Jun 26, 2026
Merged

docs: add reviewer guidance and author attestation to PR template#4106
maxrjones merged 3 commits into
zarr-developers:mainfrom
maxrjones:chore/pr-template

Conversation

@maxrjones

Copy link
Copy Markdown
Member

This PR seeks to make it clear who is responsible for the changes in a PR in a self-describing way via a simple author-attestation checkbox. It's not a perfect system as we're still figuring out how to adapt to AI assisted coding, but I think a helpful improvement.

TODO:

  • Add unit tests and/or doctests in docstrings
  • Add docstrings and API docs for any new/modified user-facing classes and functions
  • New/modified features documented in docs/user-guide/*.md
  • Changes documented as a new file in changes/
  • GitHub Actions have all passed
  • Test coverage is 100% (Codecov passes)

@github-actions github-actions Bot added the needs release notes Automatically applied to PRs which haven't added release notes label Jun 26, 2026
@maxrjones

Copy link
Copy Markdown
Member Author

oops, I pushed another change after your approval @d-v-b. Let me know if you disagree with recommending conventional commits

@maxrjones maxrjones enabled auto-merge (squash) June 26, 2026 17:57
@maxrjones maxrjones merged commit e29ddd2 into zarr-developers:main Jun 26, 2026
28 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

needs release notes Automatically applied to PRs which haven't added release notes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants