Skip to content

CI: Micromamba Workflow#210

Merged
koldunovn merged 9 commits intomainfrom
fix/ci-mamba-version
Jan 20, 2025
Merged

CI: Micromamba Workflow#210
koldunovn merged 9 commits intomainfrom
fix/ci-mamba-version

Conversation

@pgierz
Copy link
Member

@pgierz pgierz commented Jan 20, 2025

The CI action has a depcrecation, as noted here.

Migrated to use mamba-org/setup-micromamba@v2 instead of mamba-org/provision-with-micromamba in the workflow file.

@pgierz pgierz requested a review from koldunovn January 20, 2025 07:02
@pgierz
Copy link
Member Author

pgierz commented Jan 20, 2025

This fixes the issue that previously seen in #209 with the conda not getting set up correctly, but there still seems to be an issue with the requirements, at least on macOS.

@pgierz
Copy link
Member Author

pgierz commented Jan 20, 2025

OK, I don't 100% like how this was fixed (OpenSSL is apparently painful), but tests pass again.

Nothing on the actual Python side needed to be changed here, this is just for building correctly on the CI runner. If any end-users on MacOS end up with strange import errors with missing shared libraries (libcrypto.so not found), please send them to me and I can fix it once and then with their help write a better section for the documentation.

@koldunovn
Copy link
Member

Thanks a lot, Paul!

@koldunovn koldunovn merged commit 305f076 into main Jan 20, 2025
10 checks passed
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