Skip to content

pin protos - #95

Merged
NoureldinYosri merged 1 commit into
mainfrom
protos
Aug 17, 2026
Merged

pin protos#95
NoureldinYosri merged 1 commit into
mainfrom
protos

Conversation

@NoureldinYosri

Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread dev_tools/protos.txt
grpcio-tools
mypy-protobuf
grpcio-tools == 1.78.0
mypy-protobuf == 3.6.0

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cirq seems to work fine with the latest mypy-protobuf-5.1.0, https://github.com/quantumlib/Cirq/pull/8122/changes#diff-927feb3c54df3e8ce9f40ed2018d13d8821b3e000625cd3292548a371386d336. IIUC, mypy-protobuf is only used to generate the *_pb2.pyi type annotation files, but should not be a part of runtime requirements.

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

I used the version I get when I run pip list | grep mypy-protobuf in the internal venv

@pavoljuhas pavoljuhas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM with a non-blocking suggestion.

@NoureldinYosri
NoureldinYosri merged commit efc0bb4 into main Aug 17, 2026
15 checks passed
@NoureldinYosri
NoureldinYosri deleted the protos branch August 17, 2026 20:14
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