Shawnsuun commented on PR #74383: URL: https://github.com/apache/airflow/pull/74383#issuecomment-6261576605
Thanks for the review. Addressed in 2c83ba4340: - Generated bool flags now follow each field's default for every datamodel (see the inline thread), so this also fixes `pools update` and the other optional bool fields. - `dags update` fails on the client with "Provide exactly one of --is-paused / --no-is-paused or --scheduling-state." when it gets neither or both, and it leaves `None` fields out of the request body. - Tests that `--is-paused` / `--no-is-paused` reach the body as `True` / `False` were added in 4c2e3447b3, and there are matching cases for `--include-deferred` / `--no-include-deferred`. - `airflow-ctl/RELEASE_NOTES.rst`: I didn't edit it, because its header says it is populated at release time and must not be edited in PRs. I described the user-visible change in the PR description instead. Happy to add a line here if you still want one. --- Drafted-by: GitHub Copilot (Claude Opus 5.5) (no human review before posting) -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
