[infinispan-dev] PR labels

[Tristan Tarrant]

Sure: all the labels I haven't mentioned can stay.

Tristan

On 12/1/17 1:03 PM, Pedro Ruivo wrote:
> Hi,
> 
> can we keep the "changes suggested/required"?
> This would be exclusive with "ready for preview" and means someone
> review the PR and requires a reply from the author.
> 
> Cheers,
> Pedro
> 
> On 01-12-2017 08:26, Tristan Tarrant wrote:
>> Hello people,
>>
>> I'd like to rationalize the PR labels because I believe some of them are
>> useless:
>>
>> [Ready for review] - Any PR without the [Preview] label must fall under
>> this category
>> [Backport] - The burden should be on the PR owner to create relevant
>> backport PRs, not on the reviewer
>> [Wait CI Results] - PRs should only be integrated after a successful CI
>> run (or when failures can be proven to be pre-existing)
>> [Check CI Failures!] - The CI runs already add failure/success to the PR
>> status. Checking CI failures should apply to ALL PRs.
>> [On Ice] PR should be closed and reopened when relevant again.
>>
>> Comments/suggestions ?
>>
>> Tristan
>>
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/infinispan-dev
> 

-- 
Tristan Tarrant
Infinispan Lead
JBoss, a division of Red Hat