New Github Issues

Dicebot via Digitalmars-d digitalmars-d at puremagic.com
Tue Jul 29 03:42:23 PDT 2014


On Tuesday, 29 July 2014 at 07:18:37 UTC, Daniel Murphy wrote:
>> Reason why I have named it "needs-decision" (can't have 
>> whitespaces in labels afaik) is that it is not necessarily an 
>> enhancement stuff - sometimes bug fixes
>> can be also very controversial.
>
> 'Walter-blocked' is the true meaning.

It is also "Andrei-blocked" for Phobos ;)

>> Assignment does not really mean much for pure pull requests, 
>> in absence of issue tracking - only for rare cases when review 
>> is needed by certain qualified person.
>
> I wish they were rare.  In dmd at least.

Yep I am not very familiar with dmd review process, had Phobos in 
mind.

=====================

I went ahead and have added few labels for Phobos 
(https://github.com/D-Programming-Language/phobos/labels):

"blocked (awaits decision)" == "Andrei-blocked" but named a bit 
more generic just in case
"blocked by dependency" == depends on some other pull request in 
dmd / druntime
"needs work" == pull request author have not responded to review 
comments for some time
"needs review" == pull request in a good shape but needs more 
reviewer input before merging

If it makes sense probably worth making Phobos mail list 
announcement for all maintainers.


More information about the Digitalmars-d mailing list