Re: [infinispan-dev] PR labels

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(a)lists.jboss.org https://lists.jboss.org/mailman/listinfo/infinispan-dev