Skip to content

[Python] Remove deprecated type aliases from generated code for python 3.9+.#5974

Merged
andrueastman merged 9 commits intomainfrom
andrueastman/python3.9
Jan 13, 2025
Merged

[Python] Remove deprecated type aliases from generated code for python 3.9+.#5974
andrueastman merged 9 commits intomainfrom
andrueastman/python3.9

Conversation

@andrueastman
Copy link
Contributor

Partial of microsoft/kiota-python#349

Related to microsoft/kiota-python#433

Drops deprecated type hints used in python 3.9+.

@andrueastman
Copy link
Contributor Author

andrueastman commented Jan 9, 2025

Sample generation at microsoft/kiota-python#434

@andrueastman
Copy link
Contributor Author

Depends on release of microsoft/kiota-python#433

@andrueastman andrueastman changed the title Remove deprecated type aliases from generated code. [Python] Remove deprecated type aliases from generated code for python 3.9+. Jan 9, 2025
baywet
baywet previously approved these changes Jan 9, 2025
Copy link
Member

@baywet baywet left a comment

Choose a reason for hiding this comment

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

mostly good with those changes. I did leave a comment on the test generation PR. I think a regression was introduced some time ago. Not sure whether we want to address it now or as part of a separate PR.

@andrueastman andrueastman marked this pull request as ready for review January 13, 2025 07:49
@andrueastman andrueastman requested a review from a team as a code owner January 13, 2025 07:49
@andrueastman andrueastman requested a review from baywet January 13, 2025 07:49
@andrueastman
Copy link
Contributor Author

mostly good with those changes. I did leave a comment on the test generation PR. I think a regression was introduced some time ago. Not sure whether we want to address it now or as part of a separate PR.

Addressed in the PR with this change 98971c7

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

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants