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.


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.
@boring-cyborg boring-cyborg bot added area:dev-tools backport-to-v3-1-test Mark PR with this label to backport to v3-1-test branch labels Jan 21, 2026
Copy link
Contributor

@jscheffl jscheffl left a comment

Choose a reason for hiding this comment

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

Ups, cool! Thanks for fixing!

@potiuk potiuk merged commit 7214ff8 into apache:main Jan 21, 2026
70 checks passed
@potiuk potiuk deleted the run-mypy-dev-checks-also-for-scripts branch January 21, 2026 21:06
@potiuk
Copy link
Member Author

potiuk commented Jan 21, 2026

Ups, cool! Thanks for fixing!

Thanks to @mlauter for being inquisitive on slack :D

@github-actions
Copy link

Backport failed to create: v3-1-test. View the failure log Run details

Status Branch Result
v3-1-test Commit Link

You can attempt to backport this manually by running:

cherry_picker 7214ff8 v3-1-test

This should apply the commit to the v3-1-test branch and leave the commit in conflict state marking
the files that need manual conflict resolution.

After you have resolved the conflicts, you can continue the backport process by running:

cherry_picker --continue

If you don't have cherry-picker installed, see the installation guide.

potiuk added a commit to potiuk/airflow that referenced this pull request 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)

Co-authored-by: Jarek Potiuk <jarek@potiuk.com>
@potiuk
Copy link
Member Author

potiuk commented Jan 21, 2026

Backport in #60906

potiuk added a commit that referenced this pull request Jan 22, 2026
…60906)

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)
suii2210 pushed a commit to suii2210/airflow that referenced this pull request Jan 26, 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.
shreyas-dev pushed a commit to shreyas-dev/airflow that referenced this pull request Jan 29, 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.
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