You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@commons.apache.org by Behrang Saeedzadeh <be...@gmail.com> on 2016/09/11 08:31:04 UTC
Could you please checkout this PR?
Hi fellows,
I've addressed all the comments regarding this PR:
https://github.com/apache/commons-io/pull/17
Could you please kindly merge it or let me know if there's anything else
remaining that I should address before it can be merged?
--
Best regards,
Behrang Saeedzadeh
Re: Could you please checkout this PR?
Posted by Jiju Kuruvilla <jm...@gmail.com>.
Thank you, Matt!
jiju
On Mon, Sep 12, 2016 at 6:15 PM, Matt Sicker <bo...@gmail.com> wrote:
> PR = pull request. It's how people contribute patches via GitHub.
>
> On 12 September 2016 at 17:04, Jiju Kuruvilla <jm...@gmail.com>
> wrote:
>
> > hello,
> >
> > I am totally new to this. I know this may sound stupid, but what is a
> PR? I
> > joined this project to learn as well as with the intention of helping
> out.
> >
> > thanks,
> >
> > jiju
> >
> > On Sun, Sep 11, 2016 at 4:31 AM, Behrang Saeedzadeh <behrangsa@gmail.com
> >
> > wrote:
> >
> > > Hi fellows,
> > >
> > > I've addressed all the comments regarding this PR:
> > > https://github.com/apache/commons-io/pull/17
> > >
> > > Could you please kindly merge it or let me know if there's anything
> else
> > > remaining that I should address before it can be merged?
> > > --
> > > Best regards,
> > > Behrang Saeedzadeh
> > >
> >
>
>
>
> --
> Matt Sicker <bo...@gmail.com>
>
Re: Could you please checkout this PR?
Posted by Matt Sicker <bo...@gmail.com>.
PR = pull request. It's how people contribute patches via GitHub.
On 12 September 2016 at 17:04, Jiju Kuruvilla <jm...@gmail.com> wrote:
> hello,
>
> I am totally new to this. I know this may sound stupid, but what is a PR? I
> joined this project to learn as well as with the intention of helping out.
>
> thanks,
>
> jiju
>
> On Sun, Sep 11, 2016 at 4:31 AM, Behrang Saeedzadeh <be...@gmail.com>
> wrote:
>
> > Hi fellows,
> >
> > I've addressed all the comments regarding this PR:
> > https://github.com/apache/commons-io/pull/17
> >
> > Could you please kindly merge it or let me know if there's anything else
> > remaining that I should address before it can be merged?
> > --
> > Best regards,
> > Behrang Saeedzadeh
> >
>
--
Matt Sicker <bo...@gmail.com>
Re: Could you please checkout this PR?
Posted by Jiju Kuruvilla <jm...@gmail.com>.
hello,
I am totally new to this. I know this may sound stupid, but what is a PR? I
joined this project to learn as well as with the intention of helping out.
thanks,
jiju
On Sun, Sep 11, 2016 at 4:31 AM, Behrang Saeedzadeh <be...@gmail.com>
wrote:
> Hi fellows,
>
> I've addressed all the comments regarding this PR:
> https://github.com/apache/commons-io/pull/17
>
> Could you please kindly merge it or let me know if there's anything else
> remaining that I should address before it can be merged?
> --
> Best regards,
> Behrang Saeedzadeh
>