Skip to content

Conversation

@gopidesupavan
Copy link
Member

@gopidesupavan gopidesupavan commented Jul 8, 2025

related: #52997


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in airflow-core/newsfragments.

@gopidesupavan gopidesupavan self-assigned this Jul 8, 2025
@gopidesupavan gopidesupavan force-pushed the fix-mypy-task-sdk branch 2 times, most recently from dee0ba9 to d6388df Compare July 10, 2025 19:05
Copy link
Contributor

@amoghrajesh amoghrajesh left a comment

Choose a reason for hiding this comment

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

One nit, LGTM otherwise

@amoghrajesh
Copy link
Contributor

@gopidesupavan needs a rebase.

@gopidesupavan gopidesupavan merged commit a480f77 into apache:main Jul 11, 2025
76 checks passed
@gopidesupavan gopidesupavan deleted the fix-mypy-task-sdk branch July 11, 2025 13:07
stephen-bracken pushed a commit to stephen-bracken/airflow that referenced this pull request Jul 15, 2025
* Make task-sdk compatible with mypy 1.16.1

* Fixup mypy

* Fixup mypy

* Update comment for typeguard usage in secrets_masker

* Resolve comments

* Resolve pre-commit
@github-actions
Copy link

github-actions bot commented Aug 2, 2025

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

Status Branch Result
v3-0-test Commit Link

You can attempt to backport this manually by running:

cherry_picker a480f77 v3-0-test

This should apply the commit to the v3-0-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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants