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

Drop Python 3.9 support #7028

Merged
merged 2 commits into from Oct 21, 2023
Merged

Conversation

jarrodmillman
Copy link
Member

@jarrodmillman jarrodmillman marked this pull request as draft October 19, 2023 04:40
@jarrodmillman jarrodmillman marked this pull request as ready for review October 20, 2023 03:43
@jarrodmillman jarrodmillman marked this pull request as draft October 20, 2023 03:54
Copy link
Member

@dschult dschult left a comment

Choose a reason for hiding this comment

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

This looks good to me.

I looked at the changes in Python 3.10 to see what we could now use and noticed:

  • the match statement which is a super-powerful switch/case system.
  • the | operator acts with classes now (as this PR implements). The result is like a tuple combining classes in e.g. isinstance(obj, class1 | class2)
  • itertools.pairwise is now available (probably making our utility pairwise obsolete, but I haven't checked the details yet.

I'll make an issue to check whether we can/should change code due to any of these.

I also notice type annotations have been corrected in networkx/readwrite/text.py. They must have slipped into that module. I'll prepare a PR to remove those.

@jarrodmillman jarrodmillman marked this pull request as ready for review October 21, 2023 02:54
@jarrodmillman jarrodmillman merged commit ce237b7 into networkx:main Oct 21, 2023
35 checks passed
@jarrodmillman jarrodmillman deleted the drop-py39 branch October 21, 2023 02:55
@jarrodmillman jarrodmillman added this to the 3.3 milestone Oct 21, 2023
dschult pushed a commit to BrunoBaldissera/networkx that referenced this pull request Oct 23, 2023
Alex-Markham pushed a commit to Alex-Markham/networkx that referenced this pull request Oct 26, 2023
@jarrodmillman jarrodmillman mentioned this pull request Oct 27, 2023
cvanelteren pushed a commit to cvanelteren/networkx that referenced this pull request Apr 22, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging this pull request may close these issues.

None yet

3 participants