Comment by baobun

Comment by baobun 6 months ago

3 replies

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.

recursive 6 months ago

You can have both. Make sure the whole diff is what you want it to be before invoking `add .`

  • baobun 6 months ago

    Sure. I hear the pull-out method is also an effective contraceptive.

    • recursive 6 months ago

      If it works for you, I'm not going to try to talk you out of it.