Recommend miniforge conda installer in docs#395
Conversation
|
Right, this will require merging and rebasing on #393 for getting green CI. |
jakirkham
left a comment
jakirkham
left a comment
There was a problem hiding this comment.
Thanks Marvin! 🙏
This looks great 😄
With the CI PR in, updated this one to pull in the CI changes
Made some very minor tweaks below. Hope that is ok 🙂
Please let me know what you think
CONTRIBUTING.rst
Outdated
| .. note:: | ||
| If you do not have Anaconda/miniconda installed, please follow | ||
| `these instructions <https://docs.conda.io/projects/conda/en/latest/user-guide/install/>`_. | ||
| If you don't have `conda`_ installed, we recommend downloading and installing it | ||
| with the conda-forge distribution `Miniforge here`_. |
There was a problem hiding this comment.
Spotted the same issue in CONTRIBUTING.rst. The text looked the same as docs/installation.rst. So copied your text over here
| We will use conda to | ||
| `create an environment from a file | ||
| <https://docs.conda.io/projects/conda/en/latest/user-guide/tasks/manage-environments.html#creating-an-environment-from-an-environment-yml-file>`_ | ||
| `create an environment from a file`_ |
There was a problem hiding this comment.
Spotted this long in text URL. Thought it would be better to extract this. So moved it below with the 2 other URLs needed for text update
|
|
||
| If you don't have `conda`_ installed, we recommend downloading and installing it | ||
| with the conda-forge distribution `Miniforge here`_. | ||
| with the conda-forge distribution `Miniforge`_. |
There was a problem hiding this comment.
Thought the here could be dropped. So made this change throughout
|
If it looks ok, please feel free to merge Also happy to discuss as needed |
|
Thank you @jakirkham for your review and improvements here 🙏 Merging. |
|
Thanks again Marvin! 🙏 Happy to help 🙂 |
2 participants
Closes #394.