<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">Cheers,<div><br></div><div>It looks like it's really helping. For some reason I thought most of the issues were waiting on other people until I figured out what labels they needed. Then i realised most where "Ready for Review".</div><div><br></div><div>Here it is in action, BTW: <a href="https://github.com/jboss-developer/www.jboss.org/pulls">https://github.com/jboss-developer/www.jboss.org/pulls</a></div><div><br></div><div>Paul.</div><div><br><div><div>On 1 Aug 2014, at 13:16, Rafael Benevides <<a href="mailto:benevides@redhat.com">benevides@redhat.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite">Awesome initiative!<br><br>Em 8/1/14, 9:14, Pete Muir escreveu:<br><blockquote type="cite">Adding the jbossdeveloper list. (Wes, Dan, please sign up at <a href="http://lists.jboss.org">lists.jboss.org</a>)<br><br>On 1 Aug 2014, at 13:13, Paul Robinson <<a href="mailto:paul.robinson@redhat.com">paul.robinson@redhat.com</a>> wrote:<br><br><blockquote type="cite">All,<br><br>We've recently created some labels on GitHub PRs. The idea is to make it easier to see, at-a-glance what the PR is waiting upon. Although Github will allow multiple labels to be present at once, our workflow forbids it.<br><br>The workflow is quite simple:<br><br>On Hold<br>This is for WiP PRs and things that can't be merged until some external factor has been addressed. For example, <a href="https://github.com/jboss-developer/www.jboss.org/pull/336">https://github.com/jboss-developer/www.jboss.org/pull/336</a> requires a DCP outage, so we need to wait for an appropriate time. this typically replaces the "DO NOT MERGE THIS YET" text that people have been adding.<br><br>Ready for Review<br>This label means that a reviewer should take a look. Mark your PRs as this when they are ready for review or when you have completed addressing the reviewer's comments.<br><br>Address Comments<br>This means that a review round is complete and there are comments that need addressing.<br><br>Ready to Merge:<br>This indicates that a reviewer is happy for the PR to be merged. This step is skipped if the reviewer merges immediately.<br><br>Awaiting retest<br>This just indicates that a retest has been triggered. Once the test completes, it would most likely be switched to "Ready for Review". I added this label for my own convenience, it's not necessarily something you have to use.<br><br>Pete and I only set this up yesterday, so it's likely to evolve over time. Lets see how we get on with it.<br><br>Paul.<br><br>-- <br>Paul Robinson<br>JBoss Developer Team Lead (<a href="http://www.jboss.org">www.jboss.org</a>)<br><br>JBoss, a Division of Red Hat<br>Registered in England and Wales under Company Registration No. 03798903<br>Directors:Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Paul Hickey (Ireland)<br><br></blockquote><br>_______________________________________________<br>jbossdeveloper mailing list<br><a href="mailto:jbossdeveloper@lists.jboss.org">jbossdeveloper@lists.jboss.org</a><br>https://lists.jboss.org/mailman/listinfo/jbossdeveloper<br></blockquote><br>_______________________________________________<br>jbossdeveloper mailing list<br><a href="mailto:jbossdeveloper@lists.jboss.org">jbossdeveloper@lists.jboss.org</a><br>https://lists.jboss.org/mailman/listinfo/jbossdeveloper<br></blockquote></div><br><div apple-content-edited="true">
<div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br class="Apple-interchange-newline">Paul.</div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br></div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">-- <br>Paul Robinson</div><div style="color: rgb(0, 0, 0); font-family: Helvetica; font-size: 14px; font-style: normal; font-variant: normal; font-weight: normal; letter-spacing: normal; line-height: normal; orphans: auto; text-align: start; text-indent: 0px; text-transform: none; white-space: normal; widows: auto; word-spacing: 0px; -webkit-text-stroke-width: 0px; word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;">JBoss Developer Team Lead (<a href="http://www.jboss.org">www.jboss.org</a>)<br><br>JBoss, a Division of Red Hat<br>Registered in England and Wales under Company Registration No. 03798903<br>Directors:Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Paul Hickey (Ireland)</div>
</div>
<br></div></body></html>