[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: Getting help to land an open PR

Thank you, Xiaodong. That was exactly the guidance I needed. I’ve closed
https://github.com/apache/incubator-airflow/pull/3938 in favor of

- Dan

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

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)


On Fri, Oct 12, 2018 at 07:31 Deng Xiaodong <xd.deng.r@xxxxxxxxx> 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@xxxxxxxxx <danmactough@xxxxxxxxx>

> 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
>> pull request yet. Someone on Slack suggested emailing this list for
>> PR: https://github.com/apache/incubator-airflow/pull/3938
>> Thanks!