I am definitely guilt for that, but I find this approach really productive. We use small bug fixes as an opportunity to improve the code quality. Bigger PRs often introduce new features and take a lot of time, you know the other person is tired and needs to move on, so we focus on the bigger picture, requesting changes only if there is a bug or an important structural issue.

[–] breakingcups@lemmy.world 2 points 4 months ago (1 children)

So you're always behind, patching up small bits of code that don't comply with your guidelines, while letting big changes with, by deduction, worse code quality through?

[–] souperk@reddthat.com 3 points 4 months ago

Not really, we are a small team and we generally trust each other. Sure there are things that could have been better, but it's not bad either.

load more comments (5 replies)