Rules Hub
Coding Rules Library
← Back to all rules
Rule priority, scope & exceptions
Use this to align rules with the senior-level structure (P0/P1/P2, scope, exceptions/tradeoffs).
fullstack ruleP2universalStack: testing
snapshotspr-descriptionreviewtemplates
Explain every intentional snapshot change in the PR description
When a PR updates rendered-output snapshots, state in the description what changed in each snapshot and why it is intended (e.g. a page now includes a shared partial), so reviewers can verify the diff instead of asking whether it was expected.
PR: hegnar-web#4698 FCK-3550 review round 2Created: Sep 8, 2026
Bad example
Old codets
| 1 | PR description: (nothing about snapshots) |
| 2 | views/__snapshots__/preview-index.html: +<script src="https://functions.adnami.io/..."> |
Explanation (EN)
Objašnjenje (HR)
Good example
New codets
| 1 | PR description, Caveats: |
| 2 | - `preview-index.html` gains the Adnami script because preview now renders the shared `ad-tech.njk` partial; the other snapshots only move the consent preconnect one line up. |
Explanation (EN)
Objašnjenje (HR)
Notes (EN)
A reviewer asking "is this expected?" on a snapshot line means the description was missing this.