-
Notifications
You must be signed in to change notification settings - Fork 32
Lbarto/lets try again (DO NOT MERGE) #832
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
base: main
Are you sure you want to change the base?
Lbarto/lets try again (DO NOT MERGE) #832
Conversation
Signed-off-by: Lance barto <lance.barto@ibm.com>
Signed-off-by: Lance barto <lance.barto@ibm.com>
Signed-off-by: Lance barto <lance.barto@ibm.com>
Signed-off-by: Lance barto <lance.barto@ibm.com>
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.
This looks good, aside from the linter failures.
|
|
||
| # Same package with different markers should NOT raise error | ||
| c.add_constraint("baz==1.0; platform_machine != 'ppc64le'") | ||
| c.add_constraint("baz==1.1; platform_machine == 'ppc64le'") |
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.
It would be good to check here which constraint comes back out. The result should be stable since we don't have a ppc64le test job.
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.
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.
should be able to finagle some "if" statements to do that sort of assertion
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.
oh, I get it... both would have the same result
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.
Right, I couldn't remember if we had both CPU types, but I knew for sure we didn't have ppc64le. :-)

No description provided.