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

Restrict protobof<4 #1107

Merged
merged 5 commits into from
Aug 8, 2023
Merged

Restrict protobof<4 #1107

merged 5 commits into from
Aug 8, 2023

Conversation

PProfizi
Copy link
Contributor

@PProfizi PProfizi commented Aug 7, 2023

The lastest failures in the CI have been attributed to ansys-api-plaftorm-instancemanagement==1.0.0b4 because it removed the pin on protobuf<4.
Also, ansys-platform-instancemanagement==1.0.2 was pinned in requirements_test.txt but not to the latest.
This PR updates to ansys-platform-instancemanagement==1.1.1 and tests if forcing ansys-api-plaftorm-instancemanagement==1.0.0b3 temporarily solves our issues.

When using all latest pypim libraries and forcing protobuf<4: it passes

@PProfizi PProfizi added the dependencies Related to package requirements label Aug 7, 2023
@PProfizi PProfizi requested a review from anslpa August 7, 2023 16:09
@PProfizi PProfizi self-assigned this Aug 7, 2023
@codecov
Copy link

codecov bot commented Aug 7, 2023

Codecov Report

Merging #1107 (7832f47) into master (0493fe5) will increase coverage by 2.35%.
Report is 1 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1107      +/-   ##
==========================================
+ Coverage   85.12%   87.48%   +2.35%     
==========================================
  Files          81       81              
  Lines        9185     9178       -7     
==========================================
+ Hits         7819     8029     +210     
+ Misses       1366     1149     -217     

@PProfizi PProfizi changed the title Test all latest pim libraries Restrict protobof<4 Aug 8, 2023
@PProfizi PProfizi added the bug Something isn't working label Aug 8, 2023
@PProfizi PProfizi merged commit 71f940c into master Aug 8, 2023
29 of 31 checks passed
@PProfizi PProfizi deleted the maint/fix_pim branch August 8, 2023 11:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dependencies Related to package requirements
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants