Skip to content

Move operator-init exemptions list to generated/ #70361

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
shahar1 wants to merge 4 commits into from
+4 −2
Closed

Move operator-init exemptions list to generated/ #70361

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
a555ebb
Stop labelling operator-init exemption removals for backport to v3-3-…
shahar1 Jul 24, 2026
a32e7a5
Drop redundant re-ignore patterns from the exemptions exclusion
shahar1 Jul 24, 2026
666d15a
Move operator-init exemptions list to generated/
shahar1 Jul 24, 2026
775e730
Print the exemptions file path without resolving against the repo root
shahar1 Jul 24, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
0 ...ek/validate_operators_init_exemptions.txt → ...ed/validate_operators_init_exemptions.txt
View file
Open in desktop
File renamed without changes.
6 changes: 4 additions & 2 deletions scripts/ci/prek/validate_operators_init.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -28,13 +28,15 @@
from pathlib import Path
from typing import Any

from common_prek_utils import AIRFLOW_ROOT_PATH
from rich.console import Console
from rich.markup import escape

console = Console(color_system="standard", width=200)
# Pre-existing violations exempted from the checks; burn-down tracked at
# https://github.com/apache/airflow/issues/70296
EXEMPTIONS_PATH = Path(__file__).parent / "validate_operators_init_exemptions.txt"
EXEMPTIONS_FILE = "generated/validate_operators_init_exemptions.txt"
EXEMPTIONS_PATH = AIRFLOW_ROOT_PATH / EXEMPTIONS_FILE
BASE_CLASS_NAME_SUFFIXES = ("BaseOperator", "BaseSensorOperator")
# Helper callables used as template_fields values, mapped to the fields the helper injects
# on top of the explicit arguments. Injected fields are owned and assigned by the base class,
Expand Down Expand Up @@ -512,7 +514,7 @@ def main():
console.print(
f"[red]Stale exemption for {class_name} — the class has no findings anymore "
f"(or is not detected as an operator); remove its entry from "
f"{EXEMPTIONS_PATH.name}[/red]"
f"{EXEMPTIONS_FILE}[/red]"
)
return err

Expand Down