Re: [python-committers] review links

2014-02-03 Thread Gregory P. Smith
actually if your patch contains "diff -r f4377699fd47 Lib/test/test_zipimport.py" lines prior to the --- line as hg diff can emit, it'll do the right thing and show the review against the version of the file listed (or at least against it's branch head? that's an example taken from 3.3 for a patc

Re: [python-committers] review links

2014-02-03 Thread Brett Cannon
On Mon, Feb 3, 2014 at 12:03 PM, Yury Selivanov wrote: > Hello, > > Another quick infrastructure question (is this the right list, btw?): > There is always the meta bug tracker at http://psf.upfronthosting.co.za/roundup/meta/ for tracker issues. ___ pyt

Re: [python-committers] review links

2014-02-03 Thread Yury Selivanov
OK, thank you. Are you sure it's the right thing to do? I.e. a user can create a patch, ensure that it applies cleanly, and while he uploads it, someone pushes to the repo. No 'review' link, no notification. Wouldn't it make sense to always show the review link, and let the user to resolve the c

Re: [python-committers] review links

2014-02-03 Thread Zachary Ware
On Mon, Feb 3, 2014 at 11:03 AM, Yury Selivanov wrote: > Hello, > > Another quick infrastructure question (is this the right list, btw?): Core-mentorship is another good candidate. > I don't see 'review' links for some patches. Like in issue #14911, > Kristján uploaded two patches, same day, alm

Re: [python-committers] review links

2014-02-03 Thread Antoine Pitrou
On lun., 2014-02-03 at 09:16 -0800, Guido van Rossum wrote: > IIRC the review link only appears if the patch applies cleanly. And only if it applies cleanly on the default branch, IIRC :-) Regards Antoine. ___ python-committers mailing list python-c

Re: [python-committers] review links

2014-02-03 Thread Guido van Rossum
IIRC the review link only appears if the patch applies cleanly. On Mon, Feb 3, 2014 at 9:03 AM, Yury Selivanov wrote: > Hello, > > Another quick infrastructure question (is this the right list, btw?): > I don't see 'review' links for some patches. Like in issue #14911, > Kristján uploaded two pa

[python-committers] review links

2014-02-03 Thread Yury Selivanov
Hello, Another quick infrastructure question (is this the right list, btw?): I don't see 'review' links for some patches. Like in issue #14911, Kristján uploaded two patches, same day, almost same time, and I see a review link only for one of them. Screenshot: http://goo.gl/HK2oer Thanks, Yury