[Rails-core] Still trying to get pagination fixed.. STILLhavethis ActiveRecord connection helper thingy pending

[Michael Koziarski]

> Talking to yourself, first sign of madness and all that, but instead
> of hastests I propose 'tested', to indicate that the patch has unit
> tests, and that those tests have been run.  There's no reason
> submitters shouldn't add this keyword themselves, if they wish.

Tested sounds like a great addition.   I'd like to propose two more:

* trivial - for tiny patches which we could apply in 15 minutes or so.

Typically this is a typo fix, a fix for handling nil somewhere that we
currently don't.  Something which is pretty simple to review and
apply.

* notests - For patches submitted without unittests

These should also be marked XPATCH, because barring exceptional
circumstances (or trivial fixes) we don't apply patches which don't
include tests.

We can set up two more reports for these.    We'll keep the trivial
queue relatively empty, and people who want to help out can add tests
to the notests patches.

Thoughts?  Comments?

--
Cheers

Koz