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

REF back to conda #1978

Merged
merged 6 commits into from
Dec 21, 2023
Merged

REF back to conda #1978

merged 6 commits into from
Dec 21, 2023

Conversation

beckermr
Copy link
Contributor

Going back to conda w/ libmamba solver.

@xhochy
Copy link
Member

xhochy commented Dec 21, 2023

What's the reason for this? I currently prefer setup-micromamba everywhere due to its better caching mechanism.

@beckermr
Copy link
Contributor Author

Commands like mamba update --all and micromamba update --all are currently broken. See this comment: mamba-org/mamba#3033 (comment). They use the local env index information ahead of the downloaded one and so repo data patches to fix incompatibility issues are ignored. This caused us a lot of pain last night and I don't want to deal with it again.

Copy link

codecov bot commented Dec 21, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (91485a0) 68.60% compared to head (5406716) 68.74%.

❗ Current head 5406716 differs from pull request most recent head f8a8ad2. Consider uploading reports for the commit f8a8ad2 to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1978      +/-   ##
==========================================
+ Coverage   68.60%   68.74%   +0.14%     
==========================================
  Files          93       93              
  Lines        8774     8756      -18     
==========================================
  Hits         6019     6019              
+ Misses       2755     2737      -18     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@beckermr beckermr enabled auto-merge (squash) December 21, 2023 11:59
@beckermr beckermr merged commit 2df7a63 into master Dec 21, 2023
2 checks passed
@beckermr beckermr deleted the back-to-conda branch December 21, 2023 12:04
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