-
Notifications
You must be signed in to change notification settings - Fork 19
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 support for Python 3.12. #235
Conversation
Pull Request Test Coverage Report for Build 6900816393
💛 - Coveralls |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
not sure why the Pypy3 tests fail
Jens Vagelpohl wrote at 2023-11-14 02:42 -0800:
@dataflake approved this pull request.
not sure why the Pypy3 tests fail
This looks like the effect of a `print` statement
(likely in server code).
Usually, the (doc)test is likely already finished before the print
is executed; here, a special load condition likely has let it
happen during the `ZEO.DB` call.
|
@icemac, thanks for asking for review. The first patch ( The second patch ( -from ZODB.utils import p64, u64, z64, Lock, RLock
+from ZODB.utils import Lock
+from ZODB.utils import RLock
+from ZODB.utils import p64
+from ZODB.utils import u64
+from ZODB.utils import z64 to me it feels like the old version was better with higher signal to noise ratio. But this is not a strong feeling and I won't object to this change if others find it convenient. Kirill |
PyPy3 prints some texts to stdout on GHA which break the doctests. I am going to ignore these texts. |
This is the default config for imports used in all other zopefoundation projects. We could change it here but I'd prefer not to do it. |
Thank you for reviewing this PR. 😃 |
Isort imports.