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

Add Python 3.12 to test matrix. #667

Merged
merged 4 commits into from
Oct 13, 2023
Merged

Add Python 3.12 to test matrix. #667

merged 4 commits into from
Oct 13, 2023

Conversation

qkaiser
Copy link
Contributor

@qkaiser qkaiser commented Oct 13, 2023

See #666

@qkaiser qkaiser self-assigned this Oct 13, 2023
@vlaci
Copy link
Contributor

vlaci commented Oct 13, 2023

pyyaml==6.0.1 works

@qkaiser
Copy link
Contributor Author

qkaiser commented Oct 13, 2023

this is a cache issue. I don't know how we can clean all caches on workers.

Locally I had to nuke ~/.cache/pypoetry to make it work. See also https://pythontest.com/posts/2023/2023-10-02-py312-impimporter/

@vlaci
Copy link
Contributor

vlaci commented Oct 13, 2023

this is a cache issue. I don't know how we can clean all caches on workers.

Locally I had to nuke ~/.cache/pypoetry to make it work. See also https://pythontest.com/posts/2023/2023-10-02-py312-impimporter/

Caches can be managed here, although we should also change caching to use version specific keys 🤔

@qkaiser qkaiser marked this pull request as draft October 13, 2023 11:39
@qkaiser
Copy link
Contributor Author

qkaiser commented Oct 13, 2023

not a cache issue actually, I'm getting the same error locally

@qkaiser
Copy link
Contributor Author

qkaiser commented Oct 13, 2023

Not sure if my last commit is backward compatible, let's see.

…thon.

The gzip._GzipReader we're inheriting from had some changes to stay up
to date with changes in zlib library and to perform some optimization.

Specifically:

- GzipFile.read has been optimized. There is no longer a unconsumed_tail
  member to write back to padded file. This is instead handled by the
  ZlibDecompressor itself, which has an internal buffer.
- _add_read_data has been inlined, as it was just two calls.

We've adapted our own code to reflect these changes.

More info: python/cpython#97664
@qkaiser qkaiser marked this pull request as ready for review October 13, 2023 16:00
@vlaci vlaci enabled auto-merge October 13, 2023 16:11
@vlaci vlaci merged commit 745efc0 into main Oct 13, 2023
12 checks passed
@vlaci vlaci deleted the 666-python-312 branch October 13, 2023 16:11
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.

2 participants