- Issue created by @quietone
- 🇳🇿New Zealand quietone
In committer slack, awhile back, there it was said that Question 5 should be removed. Simply because committers wouldn't defer tests to a followup when a test is needed.
- Status changed to Needs review
12 days ago 3:37am 9 January 2025 - 🇳🇿New Zealand quietone
Yes, I let this one slip. There has been no discussion, so I have made the change to the policy.
- 🇺🇸United States smustgrave
So really this is around if "Is it easy for someone who did not work on the original bug report to add the test coverage in a followup issue?" belongs.
I believe agree with removing and feel that statement kinda overlaps #6 about coverage possibly missing in general. But since #5 refers to a bug I think it should 100% have test coverage. Only push back I can see if super old issues where maybe test coverage was more lenient so someone may say "it shouldn't be this hard to get a fix in" but standards change and improve.
+1 for the removal.