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.11 tests to CI #6191
Conversation
proto-plus >= 1.20.0 | ||
protobuf >= 3.15.0, < 4 | ||
protobuf >= 3.15.0 |
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.
As noted in chat, proto4 was breaking cirq_google: #5355
If that's resolved then LGTM!
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.
I think it would be better to split this to 2 PRs -
- PR1 - upgrade protobuf to 4.0
- PR2 - add Python 3.11 CI tests
In my previous attempt on PR1 I had to fix a few expected textprotos - #5951, perhaps that is still needed to pass tests for the supported Pythons.
This is a follow-up to #6181, which adds support for Python 3.11 to Cirq.