Skip to content

1176 - Initial python/pypy 3.11 refactors #1179

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

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

Carsons-Eels
Copy link
Collaborator

@Carsons-Eels Carsons-Eels commented Apr 3, 2025

What was wrong?

New pypy version was released supporting python 3.11. Basic upgrades to code were needed.

Closes #1176
Related to Issue #1144

How was it fixed?

Refactored

  • Use primitive Union type introduced by python 3.10 rather than typing.Union
  • Import assert_type from typing rather than typing_extensions

Word of Caution ⚠️

This should be merged AFTER PR #1177 ✔️

Cute Animal Picture

Put a link to a cute animal picture inside the parenthesis-->

@Carsons-Eels Carsons-Eels changed the title 1176 inital 311 refactors 1176 - initial 311 refactors Apr 3, 2025
@Carsons-Eels Carsons-Eels changed the title 1176 - initial 311 refactors 1176 - Initial 311 refactors Apr 3, 2025
@Carsons-Eels Carsons-Eels force-pushed the 1176_inital_311_refactors branch 4 times, most recently from 18059e7 to 3778acd Compare April 3, 2025 23:28
@Carsons-Eels Carsons-Eels changed the title 1176 - Initial 311 refactors 1176 - Initial python/pypy 3.11 refactors Apr 8, 2025
@Carsons-Eels Carsons-Eels marked this pull request as ready for review April 8, 2025 19:05
@Carsons-Eels
Copy link
Collaborator Author

I included the commit from PR #1177 because otherwise you can't test it locally as it requires the upgrade to py/py 3.11 to function correctly. That commit can either be dropped pre-approval, or this commit can just be merged immediately after 1177.

@Carsons-Eels Carsons-Eels force-pushed the 1176_inital_311_refactors branch from 3778acd to bf3789c Compare May 23, 2025 21:36
@Carsons-Eels Carsons-Eels changed the base branch from forks/prague to master May 23, 2025 21:37
@Carsons-Eels Carsons-Eels force-pushed the 1176_inital_311_refactors branch from bf3789c to 0bb658f Compare June 17, 2025 17:31
Copy link
Contributor

Choose a reason for hiding this comment

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

Where is this coming from?

@Carsons-Eels Carsons-Eels force-pushed the 1176_inital_311_refactors branch from 0bb658f to 8799f7a Compare June 17, 2025 18:02
@Carsons-Eels Carsons-Eels force-pushed the 1176_inital_311_refactors branch from 8799f7a to fa144d4 Compare June 18, 2025 19:40
- refactored assert_type import from typing_extensions to typing
- refactored Union to use primitive type offered by 3.10
@Carsons-Eels Carsons-Eels force-pushed the 1176_inital_311_refactors branch from fa144d4 to 9139169 Compare June 20, 2025 04:16
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.

Update Union syntax, move assert_type import,
2 participants