4 ms·
I agree to some extent. Personally, I do spend the time to teach in PRs but also approve the PR to unblock unless there's much needed changes. It unblocks peopl
by baby 4y ago
I agree to some extent. Personally, I do spend the time to teach in PRs but also approve the PR to unblock unless there's much needed changes. It unblocks people, and you quickly see if they cared about your comments if they follow up with another PR addressing them.
- bigyikes 4y agoYes, this is how you have your cake and eat it. It works in reverse too, when you’re the one being reviewed and you receive feedback you disagree with. Instead of dying on the hill and delaying your merge, just implement the feedback and then argue. You can make your point while still accommodating your teammate and moving things along. (Obviously, this practice shouldn’t apply to feedback which is truly bad)