Add airflow __main__#12889
Add airflow __main__#12889GuillaumeDesforges wants to merge 2 commits intoapache:v1-10-testfrom GuillaumeDesforges:gdesforges/1.10.14rc4-fix-__main__
Conversation
potiuk
left a comment
potiuk
left a comment
There was a problem hiding this comment.
We can always merge it to v1-10-test - then it will be released with the next release (there should be none, but you never know)!
|
We should change the |
|
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 master at your convenience, or amend the last commit of the PR, and push it with --force-with-lease. |
github-actions
bot
added
the
full tests needed
|
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*. |
| args = parser.parse_args() | ||
| args.func(args) | ||
| if __name__ == "__main__": | ||
| cli_main() No newline at end of file |
There was a problem hiding this comment.
Please ensure files have trailing new lines for consistency.
|
The Workflow run is cancelling this PR. It has some failed jobs matching ^Pylint$,^Static checks,^Build docs$,^Spell check docs$,^Backport packages$,^Provider packages,^Checks: Helm tests$,^Test OpenAPI*. |
We should exclude that file from the autoapi code generation. |
github-actions
bot
removed
the
full tests needed
kaxil
force-pushed
the
v1-10-test
branch
3 times, most recently
from
dae8ca2 to
800e630
Compare
4 participants
closes: #12888
This PR proposes to simply add a
__main__.pyfile to airflow which is an exact copy of airflow/bin/airflow.There is probably a better way to do it (see #7808 for versions 2.*+)