The GitHub Actions job "Tests (AMD)" on airflow.git/add-prek-hook-for-airflow-imports-in-shared has failed. Run started by GitHub user potiuk (triggered by potiuk).
Head commit for run: 52965a218aa3521a8d3b029107147bc21580b8ee / Jarek Potiuk <[email protected]> Improve prek hook to check for proper imports in shared distributions Builds on the boundary checks landed in #65880 by enabling the parts that still surface real violations: - Refactor task-sdk and airflow-core distribution checks through a shared `check_distribution()` helper that calls both `check_no_airflow_shared_imports` and `check_only_allowed_shared_imports`. This turns on the `airflow.sdk._shared` allowed-prefix gate inside airflow-core, which flags the residual `SecretsMasker` and friends that need to be relocated. - Add `check_devel_common_distribution()` that bans `airflow.*` imports from `devel-common/src/`. devel-common should reach airflow internals through `airflow_shared` (or move the helper into devel-common itself). - Drop TODO markers next to the two `airflow.sdk._shared.secrets_masker` imports in airflow-core's CLI (`task_command.py`, `triggerer_command.py`) pointing to the followups that will let the gate go fully green. - Add wording to the breeze provider-dependency-bump warning so it mentions moving the dependency above `# Additional devel dependencies`. The hook will fail until the residual offenders in airflow-core and devel-common are resolved — that is the intent. Reopened from #58825 after rebasing onto latest main. Generated-by: Claude Code (Opus 4.7) Report URL: https://github.com/apache/airflow/actions/runs/28941447221 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
