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

Use conda's libmamba solver #240

Closed
wants to merge 1 commit into from

Conversation

jakirkham
Copy link
Member

Now that conda-libmamba-solver ships with the installer. Opt-in to using the libmamba solver with conda.

Now that `conda-libmamba-solver` ships with the installer. Opt-in to
using the `libmamba` `solver` with `conda`.
@jakirkham jakirkham marked this pull request as draft August 22, 2023 21:27
@jakirkham
Copy link
Member Author

We can make this change here, which may be valuable just to have in the Docker images

Though we can also make this change in conda-forge-ci-setup to cover all OSes: conda-forge/conda-forge-ci-setup-feedstock#259

Or we can do both

@jaimergp
Copy link
Member

I'd say we stick to conda-forge/conda-forge-ci-setup-feedstock#259. The default will soon change anyway.

@jakirkham
Copy link
Member Author

jakirkham commented Aug 23, 2023

Yeah guess the benefit of having it here also would be the Docker image is preconfigured (as opposing to adding a step) for users that just grab the Docker image. Though have no strong feelings (given this is an easy change to make and it soon will be the default anyways)

@jaimergp
Copy link
Member

jaimergp commented Sep 7, 2023

I think I'll close here in favor of conda-forge/conda-forge-ci-setup-feedstock#275, to minimize moving parts. As you say, it will be default soon anyways, so if it breaks for non-conda-forge usage, it will be because of that, not because we rolled the change early.

@jaimergp jaimergp closed this Sep 7, 2023
@jakirkham jakirkham deleted the conda_use_libmamba branch October 2, 2023 22:18
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