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

Add python 3.12 support #483

Merged
merged 4 commits into from
Apr 15, 2024
Merged

Conversation

deronnax
Copy link
Contributor

No description provided.

@deronnax
Copy link
Contributor Author

Hmmm, the same pendulum 3.0.0 error than in #482 .

@laixintao
Copy link
Owner

Hmmm, the same pendulum 3.0.0 error than in #482 .

Seems those locales files were not packaged into pyoxidizer build https://github.com/sdispater/pendulum/tree/master/src/pendulum/locales , I am trying to fix it here #484

@deronnax
Copy link
Contributor Author

An idea of why are they needed now?

@laixintao
Copy link
Owner

An idea of why are they needed now?

Sorry, no, maybe some packaging logic changed in this version, I need more time to debug the packaging system.

@deronnax
Copy link
Contributor Author

Not a problem anymore with #492 .

@laixintao
Copy link
Owner

Wait your final check for #492 (review) , after that (whether can remove dateutil or not), we merge and release for 3.12

@laixintao laixintao merged commit 69dfcfe into laixintao:master Apr 15, 2024
22 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants