[core-workflow] Adding a "requires C" keyword?

2014-08-03 Thread Nick Coghlan
Chatting to an experienced C/C++ developer at PyCon AU today, they were worried that their *Python* skills might not be good enough to contribute to CPython. It reminded me of an idea I had a while ago, but forgot to suggest: adding a keyword specifically to indicate issues that require some C codi

Re: [core-workflow] Adding a "requires C" keyword?

2014-08-03 Thread francis
Hi Nick, I'm not particularly enamoured of that specific keyword name, so I'm interested in two specific kinds of feedback: 1. Does the extra keyword sound useful? Yes, IMHO it's useful Just complementary info about the context where the new keyword should be used is [1] (AFAIK): buildb

Re: [core-workflow] Adding a "requires C" keyword?

2014-08-03 Thread Mark Lawrence
On 03/08/2014 14:23, Nick Coghlan wrote: Chatting to an experienced C/C++ developer at PyCon AU today, they were worried that their *Python* skills might not be good enough to contribute to CPython. It reminded me of an idea I had a while ago, but forgot to suggest: adding a keyword specifically

Re: [core-workflow] Adding a "requires C" keyword?

2014-08-03 Thread anatoly techtonik
On Sun, Aug 3, 2014 at 7:02 PM, Mark Lawrence wrote: > On 03/08/2014 14:23, Nick Coghlan wrote: >> >> Chatting to an experienced C/C++ developer at PyCon AU today, they >> were worried that their *Python* skills might not be good enough to >> contribute to CPython. It reminded me of an idea I had

[core-workflow] Stage and keywords differences

2014-08-03 Thread francis
Dear core-devs, I would like to ask about the differences between (from the devguide [1]): a) stage:needs patch and keywords:patch b) stage:patch review and keywords:needs review Aren't those redundant? Why are those keywords needed? IMHO the keywords ones could be refactored out. Doesn't a issue

Re: [core-workflow] Stage and keywords differences

2014-08-03 Thread Ezio Melotti
Hi, On Sun, Aug 3, 2014 at 10:19 PM, francis wrote: > Dear core-devs, > I would like to ask about the differences between (from the devguide [1]): > a) stage:needs patch and keywords:patch Stage can only be set by "Developers", the "patch" keyword can be added by anyone and it's added automatica

Re: [core-workflow] Adding a "requires C" keyword?

2014-08-03 Thread Ezio Melotti
Hi, On Sun, Aug 3, 2014 at 4:23 PM, Nick Coghlan wrote: > Chatting to an experienced C/C++ developer at PyCon AU today, they > were worried that their *Python* skills might not be good enough to > contribute to CPython. It reminded me of an idea I had a while ago, > but forgot to suggest: adding

Re: [core-workflow] Adding a "requires C" keyword?

2014-08-03 Thread Nick Coghlan
On 4 Aug 2014 09:37, "Ezio Melotti" wrote: > > Hi, > > On Sun, Aug 3, 2014 at 4:23 PM, Nick Coghlan wrote: > > Chatting to an experienced C/C++ developer at PyCon AU today, they > > were worried that their *Python* skills might not be good enough to > > contribute to CPython. It reminded me of an

Re: [core-workflow] Stage and keywords differences

2014-08-03 Thread anatoly techtonik
On Mon, Aug 4, 2014 at 1:45 AM, Ezio Melotti wrote: > On Sun, Aug 3, 2014 at 10:19 PM, francis wrote: >> Dear core-devs, >> I would like to ask about the differences between (from the devguide [1]): >> a) stage:needs patch and keywords:patch > > Stage can only be set by "Developers", the "patch"