Skip to content

Update micromamba-installation.rst#4087

Open
KasperThystrup wants to merge 1 commit intomamba-org:mainfrom
KasperThystrup:patch-1
Open

Update micromamba-installation.rst#4087
KasperThystrup wants to merge 1 commit intomamba-org:mainfrom
KasperThystrup:patch-1

Conversation

@KasperThystrup
Copy link

Description

In installation instructions for Ubuntu
Changed curl to wget for ubuntu instalation since Curl is not installed by default. Currently it's two commands in a one-liner.

Type of Change

  • Bugfix
  • Feature / enhancement
  • [X ] CI / Documentation
  • Maintenance

Checklist

  • [X ] My code follows the general style and conventions of the codebase, ensuring consistency
  • [ X] I have performed a self-review of my code
  • I have commented my code, particularly in hard-to-understand areas
  • [ X] My changes generate no new warnings
  • I have run pre-commit run --all locally in the source folder and confirmed that there are no linter errors.
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes

Changet curl to wget for ubuntu instalation since Curl is not installed by default.
.. code:: bash

"${SHELL}" <(curl -L micro.mamba.pm/install.sh)
wget micro.mamba.pm/install.sh && bash install.sh
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would prefer if you also kept the curl version and just add the wget command as an alternative.

Also note that there are mentions on the curl line 43 and 49, so keeping it would keel the rest of the documentation in sync.

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