You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@jclouds.apache.org by Andrew Phillips <ap...@qrmedia.com> on 2014/02/02 06:48:34 UTC

Pull request review?

We've got a whole bunch of outstanding pull requests:

jclouds: 22
jclouds-labs: 4
jclouds-labs-openstack: 3
jclouds-karaf: 4
jclouds-cli: 1

Any chance of getting together for a one-day "PR jam" or so to plough  
through these and close, update or merge them?

ap

Re: Pull request review?

Posted by Andrew Phillips <ap...@qrmedia.com>.
> I managed to get to 3 today.

Thanks, Everett! I'll hopefully be able to make some time tomorrow or  
Friday...

ap

Re: Pull request review?

Posted by Everett Toews <ev...@RACKSPACE.COM>.
I managed to get to 3 today.

Everett


On Feb 3, 2014, at 9:51 AM, Andrew Phillips <ap...@qrmedia.com> wrote:

>> ones. Which brings the discussion onto the table: when do we consider
>> a PR dead? Should we close dead ones?
> 
> I would say so. They can always be reopened, if necessary. I'd say ping the authors and see if there's a response within a week or so..?
> 
> ap


Re: Pull request review?

Posted by Andrew Phillips <ap...@qrmedia.com>.
> ones. Which brings the discussion onto the table: when do we consider
> a PR dead? Should we close dead ones?

I would say so. They can always be reopened, if necessary. I'd say  
ping the authors and see if there's a response within a week or so..?

ap

Re: Pull request review?

Posted by Ignasi Barrera <ig...@gmail.com>.
Most of them haven't had activity for more than a month, so it would
be good to ping the authors and have a thought about closing dead
ones. Which brings the discussion onto the table: when do we consider
a PR dead? Should we close dead ones?

Some of those PRs can't be merged or are not passing the build. It
would be good to ping authors too.

http://jclouds-pulls.herokuapp.com/

On 2 February 2014 22:53, Ignasi Barrera <ig...@gmail.com> wrote:
> Count on me!
>
> On 2 February 2014 06:48, Andrew Phillips <ap...@qrmedia.com> wrote:
>> We've got a whole bunch of outstanding pull requests:
>>
>> jclouds: 22
>> jclouds-labs: 4
>> jclouds-labs-openstack: 3
>> jclouds-karaf: 4
>> jclouds-cli: 1
>>
>> Any chance of getting together for a one-day "PR jam" or so to plough
>> through these and close, update or merge them?
>>
>> ap

Re: Pull request review?

Posted by Ignasi Barrera <ig...@gmail.com>.
Count on me!

On 2 February 2014 06:48, Andrew Phillips <ap...@qrmedia.com> wrote:
> We've got a whole bunch of outstanding pull requests:
>
> jclouds: 22
> jclouds-labs: 4
> jclouds-labs-openstack: 3
> jclouds-karaf: 4
> jclouds-cli: 1
>
> Any chance of getting together for a one-day "PR jam" or so to plough
> through these and close, update or merge them?
>
> ap