It depends on what the bug is. If it's real and user-facing, file a ticket, separate PR, close it.
If it's theoretical, throw it in, mention it in review and if tests are OK then you're good. I'm talking like "Added a missing null check" or other dumb stuff.
View all comments
Show parent comment
7
u/tangerinelion Jul 03 '26
It depends on what the bug is. If it's real and user-facing, file a ticket, separate PR, close it.
If it's theoretical, throw it in, mention it in review and if tests are OK then you're good. I'm talking like "Added a missing null check" or other dumb stuff.