The GitHub Actions job "Tests (AMD)" on airflow.git/refactor/ci/consolidate-allow-list-manager has failed. Run started by GitHub user jason810496 (triggered by jason810496).
Head commit for run: 80d27d20b635d90edc19a79fdb19711a517490a2 / LIU ZHE YOU <[email protected]> Move check loop and parse into AllowlistManager base class The initial extraction left the scan/tighten/report loop duplicated in both hook scripts and exposed `parse` as a staticmethod that silently ignored the instance's `repo_root`. - Convert `parse` from `@staticmethod` to instance method so it always uses `self.repo_root` (fixes the latent footgun where `_parse_tracked_allowlist` called parse unbound). - Hoist the check loop into `AllowlistManager.check()` with `violation_panel_text()` (abstract) and `format_violation_details()` (overridable) hooks — both `_check_*` wrappers are now one-liners. - Fix naming: `AllowListManager` → `AllowlistManager` to match the subclass convention used throughout. - Add test coverage for `check_new_airflow_exception_usage.py`. Report URL: https://github.com/apache/airflow/actions/runs/28657969932 With regards, GitHub Actions via GitBox --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
