Skip to content

Conversation

@ashb
Copy link
Member

@ashb ashb commented Oct 29, 2020

This was merged in #11472, but for some reason that didn't run tests so
the failure wasn't noticed


^ Add meaningful description above

Read the Pull Request Guidelines for more information.
In case of fundamental code change, Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in UPDATING.md.

This was merged in apache#11472, but for some reason that didn't run tests so
the failure wasn't noticed
@ashb ashb requested review from kaxil and mik-laj October 29, 2020 17:03
@github-actions
Copy link

The PR should be OK to be merged with just subset of tests as it does not modify Core of Airflow. The committers might merge it or can add a label 'full tests needed' and re-run it to run all tests if they see it is needed!

@github-actions github-actions bot added the okay to merge It's ok to merge this PR as it does not require more tests label Oct 29, 2020
@ryw
Copy link
Member

ryw commented Oct 29, 2020

nice bot :)

@ashb ashb merged commit a7ad204 into apache:master Oct 29, 2020
michalmisiewicz pushed a commit to michalmisiewicz/airflow that referenced this pull request Oct 30, 2020
This was merged in apache#11472, but for some reason that didn't run tests so
the failure wasn't noticed
szn pushed a commit to szn/airflow that referenced this pull request Nov 1, 2020
This was merged in apache#11472, but for some reason that didn't run tests so
the failure wasn't noticed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

okay to merge It's ok to merge this PR as it does not require more tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants