Comment by baobun
2 days ago
Is not very compatible with printlog-debugging. I'd rather encourage devs to prod around as they go if it benefits them, which causes grief for either them or reviewers in the end if they've internalized what you just said.
Explicitly adding internalizes a personal review process as inherent part of the push process, instead of something you attempt to force on top later.
It's better with a collaboration workflow that limits the span of time with expected discipline, imo.
You can have both. Make sure the whole diff is what you want it to be before invoking `add .`
Sure. I hear the pull-out method is also an effective contraceptive.
If it works for you, I'm not going to try to talk you out of it.