Hacker Newsnew | past | comments | ask | show | jobs | submitlogin

My response was mostly tongue in cheek. But I'm curious... where do you work that required code reviews before every commit? Seems like an awful lot of bureaucracy. For my projects I usually set up post-commit code review systems.


I'm at a startup now. But I think you'll find most of the big shops you know about, like Google and MS, generally practice pre-commit review.

Open source projects often will do post-commit review. The review process is viewed as friction, and they don't want to block contributors due to the review process.

I personally prefer pre-commit review, although post-commit is a legitimate option (one I admittedly wasn't thinking about at the time).


So where do developers checkpoint their work until the official commit?

Either:

The smarter ones have to maintain a second source control archive (unless they are allowed to make personal branches, in which case the "public commit" is really just a merge, and probably a good thing),

or

the dumber ones just work without a net, loosing changes, and/or having to manually back out edit and test sessions that didn't go as well as hoped.

Oh, and I just realized I'm replying to the same guy about this.




Guidelines | FAQ | Lists | API | Security | Legal | Apply to YC | Contact

Search: