Skip to content

Conversation

@ammaraziz
Copy link
Contributor

Suggested reviewers:

@rpetit3
@lskatz
@boulund

Thank you!

@ammaraziz ammaraziz requested a review from a team as a code owner October 14, 2024 23:05
jfy133
jfy133 previously approved these changes Oct 15, 2024
Copy link
Contributor

@jfy133 jfy133 left a comment

Choose a reason for hiding this comment

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

Pretty much just formatting! This is great @ammaraziz ! I didn't know about the nodefaults either 👀

ammaraziz and others added 3 commits October 16, 2024 15:57
changes as suggested by reviewers.
@ammaraziz
Copy link
Contributor Author

ammaraziz commented Oct 16, 2024

TODO for myself:

  1. add info on channel leakage (see this and this and this
  2. add info bug or undocumented 'feature' of conda that implicitly adds defaults. see this and this - thanks to Robert for pointing this out.
  3. Add a brief warning in the tldr + suggestion to use minforge3.

cond -> could spelling mistake
@ammaraziz ammaraziz requested a review from a team as a code owner October 21, 2024 12:03
@ammaraziz
Copy link
Contributor Author

I've tinkered enough... I think.

Ready for editors check :)

hexylena
hexylena previously approved these changes Oct 25, 2024
Copy link
Member

@hexylena hexylena left a comment

Choose a reason for hiding this comment

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

Some small technical/formatting patches I'm applying atop but otherwise it looks good! The cover image is cute too!

This required moving it down a bit and ensuring the preview looked good too
hexylena
hexylena previously approved these changes Oct 28, 2024
adding all reviewers, add some links and reformat some sentences
jfy133
jfy133 previously approved these changes Nov 1, 2024
Copy link
Contributor

@jfy133 jfy133 left a comment

Choose a reason for hiding this comment

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

Couple of typo fixes and syntax highlighting (if not now, for if it gets implemented in the future)

- `mamba` is a drop in replacement to `conda`.
- Far more detailed information on all the different channels/distributions [can be found here.](https://bioconda.github.io/faqs.html#what-s-the-difference-between-anaconda-conda-miniconda-mamba-mambaforge-micromamba)

All the hoohah surrounds the [curated Anaconda channels](https://docs.anaconda.com/working-with-conda/reference/default-repositories/), commonly referred to as `defaults` channel. **Note: When installing `conda/miniconda` software, the `defaults` channel is added to your global channels list.** You could inadvertently be using Anaconda services without intending to.
Copy link
Contributor

Choose a reason for hiding this comment

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

@hexylena does the blog support admonitions? This might be a good sentence to put in warning

@pmenzel pmenzel merged commit 457178d into ubinfie:main Nov 1, 2024
1 check passed
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.

5 participants