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

Test all platforms with explicit example #80

Merged
merged 3 commits into from
Oct 6, 2020

Conversation

bollwyvl
Copy link
Contributor

@bollwyvl bollwyvl commented Oct 6, 2020

Looks like I failed to add a windows test for #74, and windows line endings are not picked up by the \n@EXPLICIT\n check, leading to brokenedness. Hopefully the regex ^ will figure it out...

@bollwyvl
Copy link
Contributor Author

bollwyvl commented Oct 6, 2020

The old conda on win-64 cannot deal with .conda files... presumably there is a way to make sure conda-lock doesn't do that. in the meatime, replaced the example solve with a lock of conda-lock itself from conda-forge, which doesn't generate publish .conda packages, only .tar.bz2... yet... sigh.

@bollwyvl
Copy link
Contributor Author

bollwyvl commented Oct 6, 2020

Great, that worked. Realized that the channels warning is also meaningless in the case of an explicit, so have squashed that as well...

@bollwyvl bollwyvl requested a review from goanpeca October 6, 2020 18:05
Copy link
Member

@goanpeca goanpeca left a comment

Choose a reason for hiding this comment

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

Looks good 👍

Thanks again @bollwyvl

@goanpeca goanpeca merged commit c17872f into conda-incubator:master Oct 6, 2020
@goanpeca
Copy link
Member

goanpeca commented Oct 6, 2020

So maybe now is time to make a new release?

@bollwyvl
Copy link
Contributor Author

bollwyvl commented Oct 6, 2020 via email

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