Great article on code reviews !
amitay at gmail.com
Tue Jun 3 05:19:23 MDT 2014
This raises the question "How good is Phabricator?"
Is it something worth considering for our workflow?
On Tue, Jun 3, 2014 at 3:42 AM, Simo <simo at samba.org> wrote:
> On Mon, 2014-06-02 at 10:17 -0700, Jeremy Allison wrote:
> > Hurrah, FreeBSD is now joining the club :-).
> > Some really great points in the article:
> > "But it's hard to step aside from your code once it is
> > building and running and all that is left for the world to
> > witness your creation is to "commit" it to the tree. But you
> > know what? In the vast majority of cases, nobody cares if you
> > commit your change at that exact moment, or tomorrow, or the
> > week after. It may be hard to hear this, but that pre-commit
> > "high" that rushes you to submit your code is usually
> > counterproductive and dangerous. (Been there, done that, and
> > ended up having to fix the commit soon after for stupid
> > reasons... countless times... and that is shameful.)"
> > I am soooo guilty of the above :-).
> Haha, it always happen w/o code review.
> I am guilty of that too, what could possibly go wrong with my
> magnificent 'fix', commit ... BOOM!
More information about the samba-technical