Nice idea! I am using [1] to monitor the PRs I was involved in, which does
a pretty good job, but it's annoying that it misses some updates (like the
build status, most of the time).
I have one suggestion: most PRs are ready for review the moment they are
issued, so I think that should be the default - no label required.
I would add instead a "Do not integrate yet" label :)
[1]
https://prs.paas.allizom.org/infinispan/infinispan
On Mon, Aug 4, 2014 at 11:02 AM, Radim Vansa <rvansa(a)redhat.com> wrote:
Great, thanks, Sanne! I was often in-doubt what's the actual
status of
my PR, now I will check these :)
Btw., "Ready for review" suggests that I think that it could be
integrated after a proper review. There are situations (such as my [1])
where I need some advice about the PR - should that be considered "Ready
for review", or would be some label "Advice/Review requested" fit
better? Of course, having thousand labels is not desirable, that's why I
am asking how coarse grained this should be.
Radim
[1]
https://github.com/infinispan/infinispan/pull/2585
On 08/01/2014 09:50 PM, Sanne Grinovero wrote:
> .. and from who.
>
> It's sometimes unclear which PRs are there in need for review, or
> which have been commented on and are waiting for fixes / polishing /
> rebase / denial.
>
> Hope these labels help:
>
https://github.com/infinispan/infinispan/pulls
>
> And you can bookmark them!
>
https://github.com/infinispan/infinispan/pulls?q=is%3Aopen+is%3Apr+label%...
>
> Cheers,
> Sanne
> _______________________________________________
> infinispan-dev mailing list
> infinispan-dev(a)lists.jboss.org
>
https://lists.jboss.org/mailman/listinfo/infinispan-dev
--
Radim Vansa <rvansa(a)redhat.com>
JBoss DataGrid QA
_______________________________________________
infinispan-dev mailing list
infinispan-dev(a)lists.jboss.org
https://lists.jboss.org/mailman/listinfo/infinispan-dev