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

Replace miniconda with micromamba in the Docker image #1870

Merged
merged 1 commit into from
Aug 31, 2023

Conversation

AdeelH
Copy link
Collaborator

@AdeelH AdeelH commented Aug 29, 2023

Overview

This PR replaces miniconda with micromamba in the Docker image. See the links below for some discussion of why mamba is better than conda.

Checklist

  • Added needs-backport label if PR is bug fix that applies to previous minor release
  • Ran scripts/format_code and committed any changes
  • Documentation updated if needed
  • PR has a name that won't get you publicly shamed for vagueness

Notes

N/A

Testing Instructions

  • Check if image builds correctly.

@AdeelH AdeelH added the needs-backport This PR needs to be backported to release branches label Aug 29, 2023
@codecov
Copy link

codecov bot commented Aug 31, 2023

Codecov Report

Merging #1870 (002e51c) into master (81b74a8) will decrease coverage by 0.02%.
Report is 2 commits behind head on master.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1870      +/-   ##
==========================================
- Coverage   81.62%   81.61%   -0.02%     
==========================================
  Files         190      190              
  Lines        9278     9278              
==========================================
- Hits         7573     7572       -1     
- Misses       1705     1706       +1     

see 1 file with indirect coverage changes

@AdeelH AdeelH marked this pull request as ready for review August 31, 2023 18:07
@AdeelH AdeelH merged commit b3780dd into azavea:master Aug 31, 2023
1 check passed
@AdeelH AdeelH deleted the mamba branch September 1, 2023 21:17
This was referenced Sep 5, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-backport This PR needs to be backported to release branches
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant