Re: [Codestriker-user] Feature request
Brought to you by:
sits
|
From: David S. <si...@us...> - 2004-02-10 22:24:56
|
> > The basic message is unreviewed code should never be committed to a > > "public" branch that other developers are using. > > David, > While I (in theory) agree with your last statement, we do have > multiple developers working on projects (not always, but often), and so, > they do share a branch. They are fully authorized to beat each other > with a wet noodle if one of them breaks the other's code (;>), but it > does happen. We do the reviews at the end, and sometimes the reviews > cause changes, so we need to review it again, requiring a new > codestriker topic, instead of just updating the existing one. Ok, but in terms of reviewing code going into the main branch, you are happy to review it as an ordinary diff file? What Matthew is proposing is to create a diff file that contains separate sections for each developer. -- Cheers, David |