airflow-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Dan MacTough <danmacto...@gmail.com>
Subject Re: Getting help to land an open PR
Date Fri, 12 Oct 2018 01:52:32 GMT
Thank you, Xiaodong. That was exactly the guidance I needed. I’ve closed
https://github.com/apache/incubator-airflow/pull/3938 in favor of
https://github.com/apache/incubator-airflow/pull/4039.

- Dan


On October 11, 2018 at 7:37:39 PM, Deng Xiaodong (xd.deng.r@gmail.com)
wrote:

In addition, your PR is trying to merge into branch “v1-10-stable”. This is
a branch which will not be modified anymore.

Instead, you should try to merge into “master” branch. Then committers will
cherry-pick commits from master branch for the next release, say 1.10.1.

(I believe this is also partially why your CI all failed)

XD

On Fri, Oct 12, 2018 at 07:31 Deng Xiaodong <xd.deng.r@gmail.com> wrote:

> Hi,
>
> Seems all CI tests of your PR are still failing, including Flake8 coding
> style checking.
>
> Possibly you can try to fix them first, then one of the committers may
> reply you (they respond fairly fast to “green” PRs).
>
>
> XD
>
> On Fri, Oct 12, 2018 at 05:05 danmactough@gmail.com <danmactough@gmail.com>

> wrote:
>
>> Hi, all. I opened a pull request a couple weeks ago to close
>> https://issues.apache.org/jira/browse/AIRFLOW-3046 (bug). I would really
>> like to land that fix on v1.10, but I haven't gotten any attention on
the
>> pull request yet. Someone on Slack suggested emailing this list for
help.
>>
>> PR: https://github.com/apache/incubator-airflow/pull/3938
>>
>> Thanks!
>>
>

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message