Features I'd like to see
Brad Greenlee
via web
Must Have
- Allow a project to be set up so that commits initially go to a review branch, and only after they have been reviewed and accepted do they get pushed to master.
- Request reviews by certain people. I may have a commit that has security implications, for example, and would like our head of security to review it. Or maybe it impacts the work of a particular team member. I should be able to list one or more people who I would like to review it, and have them notified by email that I've requested a review. Also, when they log in, they'll see a separate queue of commits they have been asked to review (or perhaps they just show up on top of the list of all commits, flagged in some way)
Nice to Have
- Allow "trivial commits" to be disabled. Some organizations consider every commit as needing a review, no matter if the committer considers it trivial. This should be a team or project setting.
- Allow comments on individual lines
We're using Gerrit right now, and it is actually pretty decent, albeit a bit clunky--and no GitHub integration. If you haven't already looked at it for ideas, you should.
Comments are currently closed for this discussion. You can start a new one.
Support Staff 2 Posted by rabble on 28 Apr, 2010 11:30 PM
Great! We'll add these to the list of things.
One tricky thing which we have to figure out is the right balance. There are so many possible git workflows, we want something which is fun, light weight, and easy to use. Gerrit is pretty heavy and controlling. We'd rather be more like github than perforce.
-evan
rabble closed this discussion on 28 Apr, 2010 11:30 PM.