Comment 10 for bug 373078

Revision history for this message
Martin Pool (mbp) wrote : Re: no code review status for 'merge with some tweaks'

There's a couple of things here: one being that 'tweak' is probably a more clear name for a vote than 'needs fixing'. People do seem to be confused about this overall.

The other is that for the overall status rather than votes, there is no 'tweak' or 'merge with changes' status: only 'needs review', 'in progress' and 'approved'. I think it would be useful to make that distinction so that people can see those branches that could be mindlessly/automatically merged on the one hand, and those that can be merged with no further discussion but with a bit of manual intervention on the other, as distinct from those that require a lot more work.