From: Zack R. <za...@vm...> - 2009-01-07 16:31:51
|
On Tuesday 06 January 2009 18:28:56 keithw wrote: > I'm open to ideas on improving the review process. I do read most > patches as they are posted in the commit messages & send comments on > occasion, but I'm sure there's scope for more. > > I'm not fond of a work flow where the reviewer is responsible for > pulling the patch out of email and getting it into git. I don't know if > it's a tool problem, but it seems to chew up unreasonable amounts of my > time whenever I've tried to work that way. Maybe better tools would > help, but I'm sceptical. > > Have you got a suggestion on how to integrate reviews better with the > general Mesa process? I think since we're using Git we could actually try to get the most out of it. github.com is rather impressive when it comes to services created on top of git. For example take a look at: http://github.com/blog/270-the-fork-queue they have a nice videocast showing a feature used to integrate patches. It allows you to also comment on commits in a similar way you'd do with reviewboard. I find it all quite interesting. z |