-
Notifications
You must be signed in to change notification settings - Fork 1.9k
Start deep dive docs #11059
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
Start deep dive docs #11059
Conversation
5cb6263
to
845f002
Compare
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
for more information, see https://pre-commit.ci
@kathatherine This is one of the ongoing work on the docs from the community side. |
@jezdez Excellent. This'll help me not duplicate work. I'll make sure I'm following this. |
Let me know if anything needs clarification @kathatherine! Also you will see English is not my native language so feel free to edit as you see fit :) |
Co-authored-by: Bianca Henderson <beeankha@gmail.com>
Co-authored-by: beeankha <beeankha@users.noreply.github.com>
b906271
to
41fc4db
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jaimergp There are a few TODOs left that would show up in the docs, what should we do about them?
```{admonition} Anatomy of the conda caches | ||
TODO | ||
``` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also a TODO?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I commented this out for now, not crucial to understanding the whole thing.
|
||
# Processing specs for `conda remove` | ||
|
||
<!-- WIP --> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Got some empty header sections here as well.
> TL;DR: when working with `MatchSpec` objects, | ||
> - to minimize the version change, set `MatchSpec(name=name, target=prec.dist_str())` | ||
> - to freeze the package, set all the components of `MatchSpec` individually | ||
* if the `spec` object does not have an adjective, it should be assumed it's being added to the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Something seems to be going wrong with the Markdown formatting here. I'm not sure if it affects the output. Maybe add a line beneath the blockquote?
I think we shouldn't block the 4.12.0 release on this work, if we get it done super, but not a big deal. |
@jezdez Pushed latest changes with syntax fixes and the figures from the Anaconda presentations. I see that there's now a RTD preview job, so we can review there to make sure nothing is out of place. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@jaimergp Thank you Jaime for the updates!
@kathatherine Could you do another pass over Jaime's recent changes? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two very minor things, but all else looks good to me!
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
Co-authored-by: Katherine Kinnaman <kkinnaman@anaconda.com>
pre-commit.ci autofix |
for more information, see https://pre-commit.ci
Here I will collect the deep dive guides for different aspects of conda:
conda install
from top to bottom