- Issue created by @kunal.sachdev
- Assigned to kunal.sachdev
- @kunalsachdev opened merge request.
- Issue was unassigned.
- Status changed to Needs review
over 1 year ago 2:56pm 17 February 2023 - Status changed to RTBC
over 1 year ago 7:37am 20 February 2023 - 🇧🇪Belgium Wim Leers Ghent 🇧🇪🇪🇺
Thanks for first pushing a commit with only test coverage, showing that that causes the new test coverage to fail and only then pushing the fix 👍 Makes my job as a reviewer very easy! 🙏🚀
- First commit to issue fork.
-
phenaproxima →
committed 36f5ecd3 on 3.0.x authored by
kunal.sachdev →
Issue #3342726 by kunal.sachdev, Wim Leers: UnknownPathExcluder doesn’t...
-
phenaproxima →
committed 36f5ecd3 on 3.0.x authored by
kunal.sachdev →
- Status changed to Fixed
over 1 year ago 3:49pm 21 February 2023 - 🇺🇸United States phenaproxima Massachusetts
Nice work, and I agree with Wim about the helpfulness of pushing a failing test first. Gratefully and enthusiastically merged!
Automatically closed - issue fixed for 2 weeks with no activity.
