Brett Cannon <[EMAIL PROTECTED]> added the comment:

On Tue, Sep 9, 2008 at 3:00 PM, Vinay Sajip <[EMAIL PROTECTED]> wrote:
>
> Vinay Sajip <[EMAIL PROTECTED]> added the comment:
>
> Sorry, no - whoops. It was a minor test data change which introduced the
> problem...I didn't think it was worth wasting anyone's time (I ran
> regrtest, which passed, before checkin) but if you tell me that *all*
> changes need to be reviewed,

All changes are supposed to happen with the thinking that something
like this wouldn't slip through.

> I'll make sure this gets done.

Well, I am not going to make you revert it, just make sure that future
patches you want in during the rc phase get reviewed.

> I had
> thought that only changes of a certain size/complexity needed review...
>
> Is it OK to assign my changes to you for review?

I don't necessarily have the time, so don't assign to me. Setting
"needs review" and making it a release blocker should be enough to get
someone to look it over.

-Brett

_______________________________________
Python tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue3809>
_______________________________________
_______________________________________________
Python-bugs-list mailing list
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to