Granularity
Bradley Baetz
bbaetz at student.usyd.edu.au
Thu Oct 24 08:20:42 UTC 2002
On Thu, 24 Oct 2002, David Miller wrote:
> How about if we require check-in approval now. That won't count as a
> review, because I have enough development stuff I have to worry about with
> my own patches I'm working on to spend major time reviewing things, but as
> a "yes, this feature has been approved for adding to Bugzilla". With the
> loss of a second-review in most cases, this is probably a good idea.
'eww' :)
We do require 2 reviews for schema changes, and the request stuff
certainly got at least 2 reviews, with several people looking over it. I
still don't see a problem with per-component stuff for this. The problem
with moving between products/components was noted _before_ checkin, and
allowed to go through since the current code didn't handle it, ie it
wasn't a regression.
You'll only need two screens when the flag isn't present in the new thing
_and_ is set, and by the time a cvs account request has an 'account
created' flag, one would hope that it was in the correct component. I
really don't see this as a big deal
Bradley
More information about the developers
mailing list