chore: enforce figma links in relevant prs - #734
Conversation
|
|
||
| **Design:** | ||
| - Always include a `### Design` section. | ||
| - For user-visible UI changes with an existing design, link the relevant Figma frames, using the screens map as the starting point. |
There was a problem hiding this comment.
The new guidance tells authors and agents to start with “the screens map,” but it provides no path or link, and the PR workflow only gathers the template, recent PRs, commits, and diff statistics. Authors therefore cannot reliably find existing Figma frames and may report uncertainty or no available design when a design exists. Please identify the map explicitly or add it to the workflow's gathered context. The same unclear reference also appears in .github/pull_request_template.md and AGENTS.md.
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
|
|
||
| - UI PRs with an existing design must link the relevant Figma frames under `### Design`, using the screens map as the starting point. | ||
| - UI PRs with no available design, including new features, must use `N/A — no design available.` under `### Design`. | ||
| - PRs without user-visible UI changes must use `N/A — no UI changes.` under `### Design`. |
There was a problem hiding this comment.
The new repository-wide policy requires non-UI PRs to include ### Design with N/A — no UI changes., but the existing release command generates version-bump PR bodies with Description and Screenshot / Video sections only. Release PRs will therefore bypass the newly documented requirement. Please update that generator as part of this policy rollout.
Closes #731
Description
This PR:
N/Aguidance.Linked Issues/Tasks
Design
N/A — no UI changes.
Preview
N/A — no user-visible changes.
QA Notes
Manual Tests
N/A
Automated Checks
git diff --checkpassed.