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

Address 'addDuration' warnings in python 3.12. #275

Merged
merged 2 commits into from
Dec 5, 2023

Conversation

sodul
Copy link
Collaborator

@sodul sodul commented Dec 4, 2023

This is new to python 3.12 and throws warning on every single test. This adds the new method and related property. Longer term we should probably inherit from the official TestResult class as to ensure better forward compatibility.

This depends on #274.

This is new to python 3.12 and throws warning on every single test. This adds the new method and related property. Longer term we should probably inherit from the official TestResult class as to ensure better forward compatibility.

This depends on CleanCut#274.
@coveralls
Copy link

Coverage Status

coverage: 99.918% (-0.08%) from 100.0%
when pulling 87c28a7 on clumio-oss:py3.12-warnings
into ff8d950 on CleanCut:main.

@sodul
Copy link
Collaborator Author

sodul commented Dec 4, 2023

There seem to be a race condition on a temp file in Windows with Python 3.6. I do not know if this is a flaky issue. I would recommend to follow the rest of the python community and drop support for 3.6 and 3.7 after doing a final release from main as the last version to support them, with 3.8 being the new oldest supported version.

@CleanCut
Copy link
Owner

CleanCut commented Dec 4, 2023

Can you merge my main into this branch? That should resolve the CI errors.

…warnings

* 'main' of https://github.com/CleanCut/green:
  don't try to find Python 2's readfp method
  update CI python versions
@CleanCut CleanCut merged commit 77fe786 into CleanCut:main Dec 5, 2023
15 checks passed
@sodul sodul deleted the py3.12-warnings branch December 7, 2023 05:06
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.

3 participants