_episodes/03-packaging-installing.md: moving Pip section. #79
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
brownsarahm
reviewed
May 13, 2021
_episodes/03-packaging-installing.md
Outdated
| @@ -52,7 +52,7 @@ Keep motivation as easing in and gradually scaling up to stay inclusive --> | |||
|
|
|||
| ## Pip | |||
|
|
|||
| Pip is the most common package manager for python. Pip allows you to easily install python packages locally from your computer or from an online repository like the [Python Package Index (PyPI)](https://pypi.org/). Once a package is installed with pip, you can `import` that package and use it in your own code. | |||
| Pip is the most common package manager for Python. Pip allows you to easily install Python packages locally from your computer or from an online repository like the [Python Package Index (PyPI)](https://pypi.org/). Once a package is installed with pip, you can `import` that package and use it in your own code. | |||
There was a problem hiding this comment.
is this in two places right now?
There was a problem hiding this comment.
My bad, that was a bit of an imperfect merge. I pushed ffb1ff7 that fixes it.
brownsarahm
reviewed
May 14, 2021
| changes to the package without re-installing it. Analysis code can change dramatically over time, so this is a | ||
| useful option! | ||
|
|
||
| ## Pip | ||
|
|
||
| Pip is the most common package manager for python. Pip allows you to easily install python packages locally from your computer or from an online repository like the [Python Package Index (PyPI)](https://pypi.org/). Once a package is installed with pip, you can `import` that package and use it in your own code. |
There was a problem hiding this comment.
pypi is mentioned above when introducing setup.py
I think that's why i initially had this section where it is currently
There was a problem hiding this comment.
Thanks for noticing this.
I pushed a5be1e3 that edits the phrasing to properly introduce Pip.
…hrasing. - Code review for carpentries-incubator#79. - Move the 'Relative file paths' callout block. - Edit phrasing to properly introduce Pip.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Related to #77.
Summary of changes:
Edit: I also cherry-picked 041a952 into this branch to prevent conflicts.
Best,
Vini