Skip to content

[No QA] Update BZ checklist to require comments on both PR and issue #87134

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Julesssss merged 2 commits into from
Apr 6, 2026
+3 −2
Merged

[No QA] Update BZ checklist to require comments on both PR and issue #87134

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
dc45ff2
Update BZ checklist to require comments on both PR and issue
allgandalf Apr 4, 2026
4623433
Merge branch 'Expensify:main' into update-bugzero-checklist
allgandalf Apr 6, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions contributingGuides/BUGZERO_CHECKLIST.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
# BugZero Checklist:

- [ ] **[Contributor]** The offending PR has been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.
- [ ] **[Contributor]** The offending PR and associated issue have been commented on, pointing out the bug it caused and why, so the author and reviewers can learn from the mistake.

Link to comment:
Link to the comment on the PR:
Link to the comment on the Issue:

- [ ] **[Contributor]** If the regression was CRITICAL (e.g. interrupts a core flow) A discussion in [#expensify-open-source](https://app.slack.com/client/E047TPA624F/C01GTK53T8Q) has been started about whether any other steps should be taken (e.g. updating the PR review checklist) in order to catch this type of bug sooner.

Expand Down
Loading