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 (http://pastie.caboo.se/9354), 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: