Skip to content

Commit cdad0b7

Browse files
momenbaselalvstcursoragent
committed
docs: carve out a no-visual-delta exception for screenshots
Keeps the before/after-screenshot rule for UI changes but lets PRs with no visual delta (refactors, accessibility labels, focus order) state that in the description instead of attaching meaningless images. Lands the maintainer-suggested wording from #36. Co-authored-by: Alvie Stoddard <alviestoddard@gmail.com> Co-authored-by: Cursor <cursoragent@cursor.com>
1 parent cad0055 commit cdad0b7

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

CONTRIBUTING.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -37,13 +37,13 @@ open packaging/Pesty.app
3737
- Keep it dependency-free. Prefer system frameworks (AppKit, SwiftUI, Carbon, ServiceManagement).
3838
- Match the existing style. Small, focused changes; no unrelated refactors in the same PR.
3939
- Test on both Apple Silicon and Intel where it matters (the release is universal).
40-
- UI changes: include a before/after screenshot of the strip.
40+
- UI changes: include a before/after screenshot of the strip. If the change has no visual delta (refactor, accessibility labels, focus order), say so in the PR description instead.
4141

4242
## Pull requests
4343

4444
1. Fork and branch from `main`.
4545
2. Make your change; ensure `swift build` is clean (no warnings).
46-
3. Open a PR with a clear description and screenshots for UI changes.
46+
3. Open a PR with a clear description and, for UI changes, before/after screenshots of the strip.
4747

4848
## Good first issues
4949

0 commit comments

Comments
 (0)