Skip to content

Conversation

@steveahnahn
Copy link
Contributor

Related: #59218, #59402

Passes mypy & pre-commit checks

airflow-core/src/airflow/api_fastapi
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_dag_runs.py.
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_variables.py
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_xcoms.py

@boring-cyborg boring-cyborg bot added area:API Airflow's REST/HTTP API area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Dec 22, 2025
Copy link
Member

@uranusjr uranusjr left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Somewhat random question: How close are we to flipping the pre-commit filter to a negative exclude? (Eventually we want to make it an exclusion list and remove all the exclusions)

@uranusjr uranusjr removed the backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch label Dec 22, 2025
@steveahnahn
Copy link
Contributor Author

It'd be a nice-to-have once we complete the entire migration but even then, we'd have to first deal with other client libraries that use the function .query(). that raise false positives. The concern was denoted here: #59402 (comment)

@kaxil kaxil merged commit 8f15520 into apache:main Dec 22, 2025
125 checks passed
Subham-KRLX pushed a commit to Subham-KRLX/airflow that referenced this pull request Jan 2, 2026
Related: apache#59218, apache#59402

Passes mypy & pre-commit checks

airflow-core/src/airflow/api_fastapi
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_dag_runs.py.
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_variables.py
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_xcoms.py
stegololz pushed a commit to stegololz/airflow that referenced this pull request Jan 9, 2026
Related: apache#59218, apache#59402

Passes mypy & pre-commit checks

airflow-core/src/airflow/api_fastapi
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_dag_runs.py.
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_variables.py
airflow-core/tests/unit/api_fastapi/execution_api/versions/head/test_xcoms.py
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:API Airflow's REST/HTTP API area:dev-tools

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants