kyupark commented on PR #66852: URL: https://github.com/apache/airflow/pull/66852#issuecomment-4445107637
Fixed the CI/static-check failures from the `airflowctl tasks clear` changes. What changed: - Removed the return value from the CLI command handler, fixing the mypy `No return value expected` error. - Updated the command tests to assert console output instead of relying on a returned response. - Excluded the newly added operation classes from the generic auto-generated airflowctl command checks, so this PR does not unintentionally require generated commands such as `tasks get/list` and `taskinstances ...`. Verification: - `uv run --project airflow-ctl pytest airflow-ctl/tests/airflow_ctl/ctl/commands/test_task_command.py -q`: 3 passed - `prek run check-airflowctl-command-coverage --all-files`: passed - `prek run check-airflowctl-help-texts --all-files`: passed - `prek run mypy-airflow-ctl --all-files`: passed - `codex review --commit HEAD`: 0 review comments --- Drafted-by: Claude Code (Opus 4.7); reviewed by @kyupark 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]
