Skip to content

ci: remove the manual PR screenshots emulator workflows - #250

Merged
dzid26 merged 1 commit into
mainfrom
claude/remove-pr-screenshots
Oct 16, 2026
Merged

dzid26 merged 1 commit into
mainfrom
claude/remove-pr-screenshots

Conversation

@dzid26

@dzid26 dzid26 commented Oct 16, 2026

Copy link
Copy Markdown
Collaborator

Requested by R · project thread

Part of #125.

Before: pr-screenshots.yml (manual, emulator) and pr-screenshots-comment.yml stayed as a fallback, and every Paparazzi PR comment ended with a pointer to them.

After: both workflows are deleted and the pointer is gone. Paparazzi renders the same full screens on every UI pull request, and the release smoke test and the demo canary still cover the real app on an emulator.

How: two workflow files removed, the footer line dropped from pr-paparazzi-comment.yml, AGENTS.md and ADR-0010 updated. capture-screenshots.sh, validate-pr-artifact.sh and upload-screenshot.sh stay: the smoke test, the website preview and the other comment workflows use them. The Paparazzi comment workflow runs from main, so the footer disappears after merge.


Generated by Claude Code

Paparazzi renders the same full screens on every UI pull request; the
release smoke test and the demo canary keep covering the real app on an
emulator.

Co-Authored-By: Claude Sonnet 5.5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0143ToMRfa1MjL4WnDp856Fe
@dzid26 dzid26 self-assigned this Oct 16, 2026
@dzid26
dzid26 merged commit 1902606 into main Oct 16, 2026
1 check passed
@dzid26
dzid26 deleted the claude/remove-pr-screenshots branch October 16, 2026 01:08
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