Skip to content

[No QA] Update BUGZERO_CHECKLIST.md to be more clear about where bugs are found #53409

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
tgolen merged 2 commits into from
Dec 3, 2024
Merged

[No QA] Update BUGZERO_CHECKLIST.md to be more clear about where bugs are found #53409

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
e112bbd
Update BUGZERO_CHECKLIST.md
tgolen Dec 2, 2024
2390ab0
Update BUGZERO_CHECKLIST.md
tgolen Dec 2, 2024
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: 2 additions & 3 deletions contributingGuides/BUGZERO_CHECKLIST.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,8 @@ Source of bug:
- [ ] 1z. Other:

Where bug was reported:
- [ ] 2a. Reported on production
- [ ] 2b. Reported on staging (deploy blocker)
- [ ] 2c. Reported on both staging and production
- [ ] 2a. Reported on production (eg. bug slipped through the normal regression and PR testing process on staging)
- [ ] 2b. Reported on staging (eg. found during regression or PR testing)
- [ ] 2d. Reported on a PR

@s77rt s77rt Dec 2, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
- [ ] 2d. Reported on a PR
- [ ] 2c. Reported on a PR

@s77rt s77rt Dec 2, 2024

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Unless the parser expects this to be d?

@tgolen tgolen Dec 2, 2024

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Heh, this is part of the fun... re-ordering these will mess up all the data parsing that happens on this because all previous options for 2c were "Reported on both staging and production" and it's really hard to track when this goes live (or I could just go back and update all the parsed data).

I'd like to keep it as 2d if that's OK.

- [ ] 2z. Other:

Expand Down