The GitHub Actions job "Tests (AMD)" on 
airflow.git/cleanup/airflowctl-no-op-overrides has succeeded.
Run started by GitHub user rjgoyln (triggered by rjgoyln).

Head commit for run:
13156304c715562d12b75a6ca2b561848ad9d884 / rjgoyln <[email protected]>
Remove no-op exception re-raise and parser override in airflowctl

Two pieces of airflowctl did nothing but obscure the code around them.

The `except AirflowCtlNotFoundException as e: raise e` in `get_client` is the
same no-op pattern cleaned out of the operations layer in #70376; it survived
there because it sits in client.py instead. The exception is still handled by
safe_call_command in cli_config.py, and api_client.close() still runs in the
surviving finally.

DefaultHelpParser._check_value only called super(). airflow-core's namesake
does real work (check_legacy_command), but airflow-ctl deliberately has no
legacy-command mapping, so this was a degenerate copy of it.

Report URL: https://github.com/apache/airflow/actions/runs/30701337284

With regards,
GitHub Actions via GitBox


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to