Switch CI from miniconda to micromamba#620
Conversation
|
@xylar Considering the recent changes to remove |
|
@altheaden, I think I did remove |
|
Oh, I see. yes, it needs to be removed. |
| sphinx sphinx-multiversion mock sphinx_rtd_theme | ||
| conda install -y -c ${CONDA_PREFIX}/conda-bld/ \ | ||
| mpas_tools python=${{ matrix.python-version }} \ | ||
| sphinx sphinx-multiversion mock sphinx_rtd_theme |
There was a problem hiding this comment.
This should be updated to match:
https://github.com/MPAS-Dev/MPAS-Tools/blob/master/conda_package/dev-spec.txt#L37-L39
| sphinx sphinx-multiversion mock sphinx_rtd_theme | |
| sphinx mock sphinx_rtd_theme |
|
@xylar I think I must have re-introduced it when I was rebasing onto your newest changes. I'll take care of it. |
|
No, I think I just missed it. |
|
Ah, yeah, I see what happened now. There's a call to it at the very end of the workflow where we build the docs to check for issues. Should I just remove the whole job or replace that call with something else? MPAS-Tools/.github/workflows/build_workflow.yml Lines 114 to 120 in 964681c |
|
Oops, I didn't realize which workflow this was. That last docs build command needs to be the same as in the documentation: |
|
Or maybe we can just make |
964681c to
ee60eab
Compare
|
@xylar How does this look? MPAS-Tools/.github/workflows/build_workflow.yml Lines 114 to 118 in ee60eab |
xylar
left a comment
There was a problem hiding this comment.
Yes, exactly. That looks good!
This PR switches CI to use the
setup-micromambaGitHub action instead of thesetup-minicondaGitHub action, in an effort to fix CI crashes caused by the latestminicondaversion.