From 774b417cc5603973d3c0e6fbab947cc5f2bf0c2f Mon Sep 17 00:00:00 2001 From: Mohamed Boudra Date: Fri, 31 Jul 2026 16:39:41 +0800 Subject: [PATCH] Revise PR template for clarity and structure Updated the pull request template to clarify guidelines and checklist items. --- .github/PULL_REQUEST_TEMPLATE.md | 14 +++++--------- 1 file changed, 5 insertions(+), 9 deletions(-) diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md index 7d5039aed..73ffec97c 100644 --- a/.github/PULL_REQUEST_TEMPLATE.md +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -1,19 +1,17 @@ ### Linked issue Closes # - - ### Type of change - [ ] Bug fix -- [ ] New feature (with prior issue + design alignment) +- [ ] New feature - [ ] Refactor / code improvement - [ ] Docs @@ -29,15 +27,13 @@ This is the section I read most carefully. I need to see that *you* tested this, - For UI changes: a screenshot or short video on every affected platform (mobile, web, desktop). UI claims without visual proof are not enough. - For behavior changes: the actual steps you ran, and what you observed. - For bug fixes: how you reproduced the bug before, and confirmed it's fixed after. - -AI-generated PR descriptions are fine in principle. AI-generated *verification claims* with no actual testing behind them are not, and they're easy to spot. --> ### Checklist -- [ ] One focused change. Unrelated cleanups split out. +- [ ] One focused change - [ ] `npm run typecheck` passes - [ ] `npm run lint` passes -- [ ] `npm run format` ran (Biome) -- [ ] UI changes include screenshots or video for every affected platform +- [ ] `npm run format` passes +- [ ] QA evidence - [ ] Tests added or updated where it made sense