On Fri, Mar 25, 2011 at 10:30 AM, Ilmari Karonen nospam@vyznev.net wrote:
On 03/24/2011 08:00 PM, Roan Kattouw wrote:
- We need to set a clear policy for reverting problematic revisions
(fixme's) if they aren't addressed quickly enough (again, let's say within a week). Currently we largely leave them be, but I think we should go back to something more decisive and closer to the "keep trunk runnable, or else Brion kicks your ass" paradigm and make it a bit more formal this time
This made me realize something that's only tangentially related to the existing thread, namely that we're currently using the "fixme" status in Code Review for two different kinds of commits:
1. commits that are broken and need to be fixed or reverted ASAP, and 2. commits that do more or less work, but need some followup work.
An example of the first kind of commit would be something that throws PHP fatal errors on a substantial fraction of page views. An example of the second kind might be something as minor as forgetting to update RELEASE_NOTES.
Of course, there's also a wide range of shades of gray between these two extremes, such as changes that work most of the time but break some unusual setups or use cases. Still, I do think that most "fixme" commits can be fairly cleanly assigned to one or the other of these categories, simply by asking oneself "Can I run a usable wiki with this code as it is?"
I think it might be a good idea to split these two cases into separate states. My suggestion, off the top of my head, would be to leave "fixme" for the latter and add a new "broken" status for the former.
Of course, we really shouldn't expect to have any "broken" commits in CR at any given time, since they really should be reverted and marked as such by the first person who can do so. But I think that being able to mark a commit as broken and needing a revert, even if you can't revert it yourself just then for some reason (no time, no svn access, whatever), could be a good idea. It would also make it less likely for such commits to get lost (even temporarily) among the less urgent "fixme"s.
(Ps. I'd also like to note that I very much agree with what Roan wrote in general, and I'd very much like to see us going back to something like the system he proposed. +1.)
+1 to everything Roan said and +1 to everything above.
-Chad