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

python: support Python 3.12 #5691

Merged
merged 3 commits into from Jan 18, 2024
Merged

Conversation

grondo
Copy link
Contributor

@grondo grondo commented Jan 18, 2024

This PR adds a few minor fixes required for Python 3.12 as identified by @vsoch, plus a test failure under the same (discovered in the conda-forge build)

grondo and others added 3 commits January 18, 2024 11:15
Problem: Building flux-core with Python 3.12 results in some SyntaxError
warnings about invalid escape sequences.

Use raw strings where necessary to avoid the warnings.

Fixes flux-framework#5674

Co-authored-by: vsoch <vsoch@users.noreply.github.com>
Problem: Man pages are not rebuilt when doc/manpages.py is edited.

Add manpages.py to the prerequesites for MAN_FILES in the Makefile.
Problem: The python/t0012-future.py test uses assertDictContainsSubset
which has been deprecated for awhile and was removed in Python 3.12.

Update the test to use a custom is_subset() function instead of
unittest's assertDictContainsSubset().
Copy link
Member

@chu11 chu11 left a comment

Choose a reason for hiding this comment

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

LGTM

@vsoch
Copy link
Member

vsoch commented Jan 18, 2024

Thank you!

@grondo
Copy link
Contributor Author

grondo commented Jan 18, 2024

Thanks! I've set MWP.

Copy link

codecov bot commented Jan 18, 2024

Codecov Report

Merging #5691 (11d1a24) into master (74bd805) will decrease coverage by 28.02%.
The diff coverage is n/a.

Additional details and impacted files
@@             Coverage Diff             @@
##           master    #5691       +/-   ##
===========================================
- Coverage   83.46%   55.44%   -28.02%     
===========================================
  Files         485      438       -47     
  Lines       82885    75286     -7599     
===========================================
- Hits        69177    41744    -27433     
- Misses      13708    33542    +19834     

see 409 files with indirect coverage changes

@mergify mergify bot merged commit f1feae8 into flux-framework:master Jan 18, 2024
33 of 34 checks passed
@grondo grondo deleted the python3.12 branch January 18, 2024 21:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants