Re: [Github] PR review

2018-04-04 Thread Juan P
Thanks Javi Roman

On 4 April 2018 at 09:53, Javi Roman  wrote:

> Juan P.
>
> The PR was already merged here [1]
>
>
> [1] https://github.com/apache/incubator-myriad/commit/
> 2f4ad0d6bf277548cc0f5bbdde0d18881b8a857b
> --
> Javi Roman
>
> Twitter: @javiromanrh
> GitHub: github.com/javiroman
> Linkedin: es.linkedin.com/in/javiroman
> Big Data Blog: dataintensive.info
>
>
> On Sat, Feb 10, 2018 at 1:41 PM, John D. Ament 
> wrote:
> > Hi,
> >
> > I just want to point out, Mohit is a committer on the project.  I'm not
> sure what your workflow is, but considering the lack of developers on the
> project maybe it makes sense to recommend people to commit directly to the
> repository?
> >
> > John
> >
> > On 2018/02/07 09:07:18, Juan P  wrote:
> >> Please, can anyone merge this PR?
> >>
> >> https://github.com/apache/incubator-myriad/pull/104
> >>
> >> It is very important/necessary for the proper functioning of Travis.
> >>
> >>
> >> Thanks
> >>
>


Re: [Github] PR review

2018-04-04 Thread Javi Roman
Juan P.

The PR was already merged here [1]


[1] 
https://github.com/apache/incubator-myriad/commit/2f4ad0d6bf277548cc0f5bbdde0d18881b8a857b
--
Javi Roman

Twitter: @javiromanrh
GitHub: github.com/javiroman
Linkedin: es.linkedin.com/in/javiroman
Big Data Blog: dataintensive.info


On Sat, Feb 10, 2018 at 1:41 PM, John D. Ament  wrote:
> Hi,
>
> I just want to point out, Mohit is a committer on the project.  I'm not sure 
> what your workflow is, but considering the lack of developers on the project 
> maybe it makes sense to recommend people to commit directly to the repository?
>
> John
>
> On 2018/02/07 09:07:18, Juan P  wrote:
>> Please, can anyone merge this PR?
>>
>> https://github.com/apache/incubator-myriad/pull/104
>>
>> It is very important/necessary for the proper functioning of Travis.
>>
>>
>> Thanks
>>


Re: [Github] PR review

2018-02-10 Thread John D . Ament
Hi,

I just want to point out, Mohit is a committer on the project.  I'm not sure 
what your workflow is, but considering the lack of developers on the project 
maybe it makes sense to recommend people to commit directly to the repository?

John

On 2018/02/07 09:07:18, Juan P  wrote: 
> Please, can anyone merge this PR?
> 
> https://github.com/apache/incubator-myriad/pull/104
> 
> It is very important/necessary for the proper functioning of Travis.
> 
> 
> Thanks
> 


[Github] PR review

2018-02-07 Thread Juan P
Please, can anyone merge this PR?

https://github.com/apache/incubator-myriad/pull/104

It is very important/necessary for the proper functioning of Travis.


Thanks