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 micromamba for CI builds #812

Merged
merged 6 commits into from
Jul 13, 2023
Merged

Use micromamba for CI builds #812

merged 6 commits into from
Jul 13, 2023

Conversation

pont-us
Copy link
Member

@pont-us pont-us commented Feb 15, 2023

Fixes #785 .

Update xcube's AppVeyor and GitHub workflow configurations to install and use micromamba rather than installing mamba via conda. See #785 for more details.

Checklist:

  • Add unit tests and/or doctests in docstrings n/a
  • Add docstrings and API docs for any new/modified user-facing classes and functions n/a
  • New/modified features documented in docs/source/* n/a
  • Changes documented in CHANGES.md
  • GitHub CI passes
  • AppVeyor CI passes
  • Test coverage remains or increases (target 100%)

@pont-us pont-us marked this pull request as ready for review July 13, 2023 09:29
@pont-us pont-us marked this pull request as draft July 13, 2023 09:40
@codecov-commenter
Copy link

Codecov Report

Patch coverage has no change and project coverage change: -0.01 ⚠️

Comparison is base (83e7bb0) 93.18% compared to head (acddefb) 93.18%.

❗ Your organization is not using the GitHub App Integration. As a result you may experience degraded service beginning May 15th. Please install the Github App Integration for your organization. Read more.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #812      +/-   ##
==========================================
- Coverage   93.18%   93.18%   -0.01%     
==========================================
  Files         428      428              
  Lines       33922    33922              
==========================================
- Hits        31611    31609       -2     
- Misses       2311     2313       +2     

see 1 file with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

@pont-us pont-us marked this pull request as ready for review July 13, 2023 11:09
@pont-us pont-us requested review from TejasMorbagal and removed request for TonioF July 13, 2023 12:57
@pont-us pont-us merged commit 0e5fc2c into master Jul 13, 2023
1 check passed
@pont-us pont-us deleted the pont-785-micromamba branch July 14, 2023 09:41
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.

Use micromamba for CI builds, rather than miniconda + mamba
4 participants