Skip to content

Document screenshot requirements for UI pull requests - #1749

Merged
gjkim42 merged 1 commit into
mainfrom
docs/ui-pr-screenshot-guidance
Sep 16, 2026
Merged

gjkim42 merged 1 commit into
mainfrom
docs/ui-pr-screenshot-guidance

Conversation

@gjkim42

@gjkim42 gjkim42 commented Sep 16, 2026 •

Copy link
Copy Markdown
Collaborator

What type of PR is this?

/kind docs

What this PR does / why we need it:

Adds UI review requirements to AGENTS.md: PR descriptions must include clearly labeled before and after screenshots captured in the same view and state, with recordings when needed to show interaction changes. Media used only in PR descriptions must be uploaded as PR attachments instead of committed to the repository.

Which issue(s) this PR is related to:

N/A

Special notes for your reviewer:

Documentation-only change. Validation: git diff --check passed. No runtime tests were needed.

Does this PR introduce a user-facing change?

NONE

Summary by cubic

Adds UI review requirements to AGENTS.md so PR descriptions for UI changes must include clearly labeled before and after screenshots captured in the same view and state, with recordings when needed to show interaction changes. Media used only in PR descriptions must be uploaded as PR attachments instead of committed to the repository.

Written for commit 9ece6e4. Summary will update on new commits.

Review in cubic

@cubic-dev-ai cubic-dev-ai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

No issues found across 1 file

Re-trigger cubic

@gjkim42
gjkim42 added this pull request to the merge queue Sep 16, 2026
Merged via the queue into main with commit 0c77e73 Sep 16, 2026
23 checks passed
@gjkim42
gjkim42 deleted the docs/ui-pr-screenshot-guidance branch September 16, 2026 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant