-
Notifications
You must be signed in to change notification settings - Fork 9
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
feat: solvability implementation for rattler-build #58
Conversation
Alright, this works! :) Will have to come up with some unsovlable ones for a test. |
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.
The suppress_output()
context needs to be added back in the correct spots.
I restored the output supression. I think from my end this doesn't really need more work. |
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.
Cool. A few more items.
-
The print statements need to be changed to print_debug
-
we need a test with an unsolvable recipe that shows what the output looks like.
The error looks something like:
I think that should be fine - wdyt? |
Also removed print statements / replaced with |
No description provided.