[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