Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feature: add support to python 3.13 #366

Merged

Conversation

henriquemeca
Copy link
Contributor

@henriquemeca henriquemeca commented Feb 13, 2025

  • If this is non-trivial feature, paste the links/URLs to the design doc.
  • Update the documentation to include this new feature.
  • Tests(including UT, IT, E2E) are added to verify the new feature.
  • If it's UI related, attach the screenshots below.
  • I have rebuilt the Configuration.md documentation by running make doc-gen

The feature requirements where not fullfilled, but i leave this PR a suggested solution for enabling the project to work with higher python versions. Feel free to change to the code if needed.

The support for higher python versions is achieved by fixing an incompability issue with the new importlib module, changed in python version 3.12

@wu-sheng wu-sheng requested a review from kezhenxu94 February 13, 2025 12:14
@wu-sheng wu-sheng added this to the 1.2.0 milestone Feb 13, 2025
@henriquemeca henriquemeca changed the title fix: python import module incompatibility feature: add support to python 3.13 Feb 13, 2025
Henrique Brito and others added 2 commits February 13, 2025 09:20
@@ -63,7 +63,7 @@ gen-basic:
.PHONY: install
install: gen-basic
python3 -m pip install --upgrade pip
python3 -m pip install .[all]
python3 -m pip install -e .[all]

Choose a reason for hiding this comment

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

suggestion: Thinking about this parameter, maybe is not necesssary

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It adds support for editable mode, its a suggestion to improve development experience. But it's not a problem to remove it 👍

@kezhenxu94 kezhenxu94 added feature New feature core labels Feb 13, 2025
Copy link
Member

@kezhenxu94 kezhenxu94 left a comment

Choose a reason for hiding this comment

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

Thanks. I didn't have time to add tests for the new version or adapt the build script for the new version. From a rough glance it requires way more changes than I expected. If you want to add, feel free to file another pull request. I will merge this since it doesn't affect previous versions

@kezhenxu94 kezhenxu94 merged commit f6bbda3 into apache:master Feb 13, 2025
62 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core feature New feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants