Trac Triage with THE MAGGOT (Time To Clean Your Patches)

Evening,

Alright folks, as Jeremy Kemper mentioned I'm working on a set of "triage" scripts for the Rails trac using RFuzz to go through all the tickets and clean them out. I've talked this over with the core guys, and they're behind it.

I'm calling this set of scripts "THE MAGGOT" thanks to hasmanyjosh. Basically THE MAGGOT (all caps) will be run against the trac tickets periodically and will auto-reject any tickets that don't meet certain criteria.

The goal of this is to keep the rails patches and trac tickets focused and actionable. Your tickets won't actually go away, they'll just be flagged "wontfix" with a terse little message about what's wrong with it.

Everyone should take a look at the list and feel free to comment on the results. REMEMBER NOTHING HAS BEEN DONE YET. We're just exploring it.

Ok, first, here's the writeup about the criteria and the list of tickets on the chopping block:

http://pastie.caboo.se/9354

And here's the ticket listings:

http://dev.rubyonrails.org/report/16 http://dev.rubyonrails.org/report/18 http://dev.rubyonrails.org/report/11 http://dev.rubyonrails.org/report/3

WHAT TO DO

* If you have a ticket in the list (Parked at Loopia), look at what it says is wrong and fix that. * If your patch is really old, just mark it "resolved: wontfix". * If your patch still applies, but maybe isn't tested with the latest 1.1.6 then retest, regenerate, and resubmit. * Finally, 1.1.6 isn't listed as a version yet, so if your patch applies to 1.1.6 then just set it to 1.1.1.

Alright, thanks a bunch folks and enjoy your weekend.

What??? Why do that when you could just move trac to google groups, email everyone that has submitted a ticket the entire trac database to look at and make sure they don't have a bad ticket, then post it on some obscure webpage somewhere and go have a pizza?

Sorry, couldn't resist:) That cleanup looks pretty reasonable. What about tickets that were submitted without a patch for older versions that address a problem that still exists?

These are just the tickets we hope to expose: old, unattended issues which are little better than closed already.

jeremy

Yes, it'll be trademarked so watch out. My lawyer's a badass.

Zed Shaw wrote:

* If your patch still applies, but maybe isn't tested with the latest 1.1.6 then retest, regenerate, and resubmit. * Finally, 1.1.6 isn't listed as a version yet, so if your patch applies to 1.1.6 then just set it to 1.1.1.

Please, please can someone add the missing versions in Trac?

It makes no sense for people to test against 1.1.6, and provide a patch to 1.1.6, but then have to declare it to be for 1.1.1.

   Justin

Your lawyer might want to check if The Melvins don't already have it trademarked :slight_smile:

http://rubyurl.com/pQP

Shane Vitarana shanesbrain.net

Just following up on this - is there some reason 1.1.1 is the latest version in trac? Shouldn't there also be an "edge" version, for defects that only exist in trunk?

- Rob

Yes - there should be 1.1.2 - 1.1.6 and 1.2.edge. They’re sitting in a basecamp todo begging to be checked off :slight_smile:

jeremy

Jeremy Kemper wrote: