[infinispan-dev] PR labels
Pedro Ruivo
pedro at infinispan.org
Fri Dec 1 07:03:02 EST 2017
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
>
More information about the infinispan-dev
mailing list