Skip to content

ci: switch to micromamba#1401

Merged
ianna merged 13 commits intomainfrom
ariostas/pin_ci_mamba
Mar 12, 2025
Merged

ci: switch to micromamba#1401
ianna merged 13 commits intomainfrom
ariostas/pin_ci_mamba

Conversation

@ariostas
Copy link
Copy Markdown
Member

@ariostas ariostas commented Mar 6, 2025

I'm pinning the Mamba version that the CI uses so that it stops being so unreliable. The issue was reported in conda-incubator/setup-miniconda#392 but it's been a while and it hasn't been fixed. The suggestion they have there is to pin the Mamba version, so that's what I'm doing. I'll run the tests a few times to make sure that this fix works and then I'll mark it as ready for review.

Closes #1391

@ariostas
Copy link
Copy Markdown
Member Author

ariostas commented Mar 7, 2025

I'm not really sure why this is not working. It seems like up to 2.0.5 it's broken on Windows, and 2.0.6 should be fixed (at least from reading the release notes). However, it's not able to find 2.0.6 even though it's on conda-forge.

@ariostas ariostas changed the title ci: pin mamba version for now ci: switch to micromamba Mar 12, 2025
@ariostas ariostas marked this pull request as ready for review March 12, 2025 14:29
@ariostas
Copy link
Copy Markdown
Member Author

This is ready for review now. I ended up switching to micromamba and it seems to be stable now. I ran the tests a few times and there were no issues with mamba (there were a couple of unrelated issues, probably from me rerunning tests so much).

@ariostas ariostas requested review from henryiii and ianna March 12, 2025 14:33
Copy link
Copy Markdown
Member

@ianna ianna left a comment

Choose a reason for hiding this comment

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

@ariostas - Great! Looks good to me. Thanks.

@ianna ianna merged commit 600d986 into main Mar 12, 2025
93 of 96 checks passed
@ianna ianna deleted the ariostas/pin_ci_mamba branch March 12, 2025 21: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.

CI is being fairly unreliable

3 participants