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

Fix minimal required protobuf #1783

Closed
wants to merge 1 commit into from
Closed

Conversation

PProfizi
Copy link
Contributor

@PProfizi PProfizi commented Oct 1, 2024

Due to runtime_version.py shown as missing for previous versions.
See here for the CI run with protobuf<5.27.0.

Due to runtime_version.py shown as missing for previous versions.
@PProfizi PProfizi self-assigned this Oct 1, 2024
@PProfizi PProfizi added the dependencies Related to package requirements label Oct 1, 2024
@PProfizi
Copy link
Contributor Author

PProfizi commented Oct 2, 2024

@rafacanton @BClappe after discussing with the PyAnsys team, it seems that we actually need to force a maximum version on the library used to build ansys-grpc-dpf as other pyansys libraries are not ready to bump to protobuf>=5

@PProfizi PProfizi closed this Oct 2, 2024
@PProfizi PProfizi deleted the dependencies/minimal_protobuf branch November 28, 2024 14:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Related to package requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants