[jbossdeveloper] GitHub Usage

Paul Robinson paul.robinson at redhat.com
Fri Aug 1 08:17:58 EDT 2014


Cheers,

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".

Here it is in action, BTW: https://github.com/jboss-developer/www.jboss.org/pulls

Paul.

On 1 Aug 2014, at 13:16, Rafael Benevides <benevides at redhat.com> wrote:

> Awesome initiative!
> 
> Em 8/1/14, 9:14, Pete Muir escreveu:
>> Adding the jbossdeveloper list. (Wes, Dan, please sign up at lists.jboss.org)
>> 
>> On 1 Aug 2014, at 13:13, Paul Robinson <paul.robinson at redhat.com> wrote:
>> 
>>> All,
>>> 
>>> 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.
>>> 
>>> The workflow is quite simple:
>>> 
>>> On Hold
>>> This is for WiP PRs and things that can't be merged until some external factor has been addressed. For example, https://github.com/jboss-developer/www.jboss.org/pull/336 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.
>>> 
>>> Ready for Review
>>> 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.
>>> 
>>> Address Comments
>>> This means that a review round is complete and there are comments that need addressing.
>>> 
>>> Ready to Merge:
>>> This indicates that a reviewer is happy for the PR to be merged. This step is skipped if the reviewer merges immediately.
>>> 
>>> Awaiting retest
>>> 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.
>>> 
>>> Pete and I only set this up yesterday, so it's likely to evolve over time. Lets see how we get on with it.
>>> 
>>> Paul.
>>> 
>>> -- 
>>> Paul Robinson
>>> JBoss Developer Team Lead (www.jboss.org)
>>> 
>>> JBoss, a Division of Red Hat
>>> Registered in England and Wales under Company Registration No. 03798903
>>> Directors:Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Paul Hickey (Ireland)
>>> 
>> 
>> _______________________________________________
>> jbossdeveloper mailing list
>> jbossdeveloper at lists.jboss.org
>> https://lists.jboss.org/mailman/listinfo/jbossdeveloper
> 
> _______________________________________________
> jbossdeveloper mailing list
> jbossdeveloper at lists.jboss.org
> https://lists.jboss.org/mailman/listinfo/jbossdeveloper


Paul.

-- 
Paul Robinson
JBoss Developer Team Lead (www.jboss.org)

JBoss, a Division of Red Hat
Registered in England and Wales under Company Registration No. 03798903
Directors:Michael Cunningham (US), Charles Peters (US), Matt Parson (US), Paul Hickey (Ireland)

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.jboss.org/pipermail/jbossdeveloper/attachments/20140801/51c7c13c/attachment-0001.html 


More information about the jbossdeveloper mailing list