Skip to content

Conversation

@dstandish
Copy link
Contributor

No description provided.

@dstandish dstandish marked this pull request as ready for review February 9, 2022 23:10
@dstandish dstandish requested review from uranusjr and removed request for XD-DENG, ashb and kaxil February 9, 2022 23:10
Copy link
Member

@jedcunningham jedcunningham left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Weird that it can't figure those out 🤷‍♂️

@github-actions github-actions bot added the full tests needed We need to run full set of tests for this PR to merge label Feb 9, 2022
@github-actions
Copy link

github-actions bot commented Feb 9, 2022

The PR most likely needs to run full matrix of tests because it modifies parts of the core of Airflow. However, committers might decide to merge it quickly and take the risk. If they don't merge it quickly - please rebase it to the latest main at your convenience, or amend the last commit of the PR, and push it with --force-with-lease.

@dstandish dstandish merged commit acad846 into apache:main Feb 9, 2022
@dstandish dstandish deleted the try-fix-mypy branch February 9, 2022 23:37
@dstandish
Copy link
Contributor Author

Weird that it can't figure those out 🤷‍♂️

i know right?

ferruzzi pushed a commit to ferruzzi/airflow that referenced this pull request Feb 11, 2022
@jedcunningham jedcunningham added the changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) label Feb 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog:skip Changes that should be skipped from the changelog (CI, tests, etc..) full tests needed We need to run full set of tests for this PR to merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants