Skip to content

Conversation

@potiuk
Copy link
Member

@potiuk potiuk commented Jan 21, 2026

Mypy Checks in CI have not been run on "scripts" folder - only on dev folder and if a mypy error creeped-in, it remained undetected unless someone run pre-push mypy check that instead of folders run it on individual files.

This PR fixes the errors that creeped in and fixes the CI so that mypy-dev also checks scripts.
(cherry picked from commit 7214ff8)


Was generative AI tooling used to co-author this PR?
  • Yes (please specify the tool below)

  • Read the Pull Request Guidelines for more information. Note: commit author/co-author name and email in commits become permanently public when merged.
  • For fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
  • When adding dependency, check compliance with the ASF 3rd Party License Policy.
  • For significant user-facing changes create newsfragment: {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

Mypy Checks in CI have not been run on "scripts" folder - only on
dev folder and if a mypy error creeped-in, it remained undetected
unless someone run `pre-push` mypy check that instead of folders
run it on individual files.

This PR fixes the errors that creeped in and fixes the CI so that
mypy-dev also checks scripts.
(cherry picked from commit 7214ff8)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@potiuk potiuk merged commit f72116c into apache:v3-1-test Jan 22, 2026
63 checks passed
@potiuk potiuk deleted the backport-7214ff8-v3-1-test branch January 22, 2026 16:05
@ephraimbuddy ephraimbuddy added this to the Airflow 3.1.7 milestone Jan 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants