You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@airflow.apache.org by Sumit Maheshwari <su...@gmail.com> on 2016/08/11 05:59:21 UTC

Long pending PR

Hello guys,

There has been a PR long pending for review, can some one please take a
look on it and merge it?

https://github.com/apache/incubator-airflow/pull/1682/files

There is a very small change in core files and all changes are in contrib
section only, which too are mainly for fixing pep8 warnings only.


Thanks,
Sumit

Re: Long pending PR

Posted by Sumit Maheshwari <su...@gmail.com>.

Hi Sid,

Can you merge this now?


Thanks,
Sumit

On Thu, Aug 11, 2016 at 11:43 AM, siddharth anand <sa...@apache.org> wrote:

> Looking at it now!
>
>
> -s
>
> On Wed, Aug 10, 2016 at 10:59 PM, Sumit Maheshwari <sumeet.manit@gmail.com
> >
> wrote:
>
> > Hello guys,
> >
> > There has been a PR long pending for review, can some one please take a
> > look on it and merge it?
> >
> > https://github.com/apache/incubator-airflow/pull/1682/files
> >
> > There is a very small change in core files and all changes are in contrib
> > section only, which too are mainly for fixing pep8 warnings only.
> >
> >
> > Thanks,
> > Sumit
> >
>

Re: Long pending PR

Posted by siddharth anand <sa...@apache.org>.

Looking at it now!


-s

On Wed, Aug 10, 2016 at 10:59 PM, Sumit Maheshwari <su...@gmail.com>
wrote:

> Hello guys,
>
> There has been a PR long pending for review, can some one please take a
> look on it and merge it?
>
> https://github.com/apache/incubator-airflow/pull/1682/files
>
> There is a very small change in core files and all changes are in contrib
> section only, which too are mainly for fixing pep8 warnings only.
>
>
> Thanks,
> Sumit
>