Skip to content
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

Clarify docs #8280

Closed
wants to merge 8 commits into from
Closed

Clarify docs #8280

wants to merge 8 commits into from

Conversation

temeddix
Copy link

@temeddix temeddix commented Aug 8, 2023

Pull Request Check List

I've discovered that there is plenty of room for improvement in the documentation. Considerable portion of Python coders are not professional programmers, so we need more guides about environment variables and PATH. Also, More easier description and crucial information about using Poetry on Windows and Python from MS Store were needed.

I am very much experienced with conda and pip, but the first impression of Poetry was quite frustrating. Adding this information to the docs will reduce the barrier considerably.

  • Added tests for changed code.
  • Updated documentation for changed code.

@temeddix temeddix changed the title Clarify uninstallation procedure Clarify docs about using Poetry on Windows Aug 8, 2023
@temeddix
Copy link
Author

temeddix commented Aug 8, 2023

If this PR is merged, I will try to provide further improvements with Poetry experience for those using Python from Microsoft Store. This issue has been present for a long time since #5331.

@temeddix temeddix changed the title Clarify docs about using Poetry on Windows Clarify docs about using Poetry on Windows and Python from MS Store Aug 8, 2023
@temeddix temeddix changed the title Clarify docs about using Poetry on Windows and Python from MS Store Clarify docs Aug 8, 2023
@temeddix
Copy link
Author

temeddix commented Aug 21, 2023

Isn't there anybody to review this?

@radoering radoering added the impact/docs Contains or requires documentation changes label Aug 25, 2023
@github-actions
Copy link

github-actions bot commented Aug 25, 2023

Deploy preview for website ready!

✅ Preview
https://website-oa6a94ptp-python-poetry.vercel.app

Built with commit b5e0d7c.
This pull request is being automatically deployed with vercel-action

docs/_index.md Outdated Show resolved Hide resolved
docs/_index.md Outdated Show resolved Hide resolved
@@ -79,7 +84,7 @@ By default, Poetry is installed into a platform and user-specific directory:

- `~/Library/Application Support/pypoetry` on MacOS.
- `~/.local/share/pypoetry` on Linux/Unix.
- `%APPDATA%\pypoetry` on Windows.
- `%APPDATA%\pypoetry` on Windows (If Python is installed via web installer).
Copy link
Member

Choose a reason for hiding this comment

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

Where is it installed if Python is installed through the Microsoft Store and POETRY_HOME has not been set? Or does installation just fail?

docs/_index.md Show resolved Hide resolved
Comment on lines +122 to +134
The way to add Poetry to your PATH differs by platform.
For the changes in PATH to take effect, you should always restart the terminal or re-open the window.

- On Linux and macOS, open (or create) the shell file.
For example, if Bash is your default shell, you should be editing `$HOME/.bashrc` file.
The filepath and filename will be different depending on the default shell.
Typing `echo $SHELL` in your Terminal will tell you which shell you’re using.
Add `export PATH="$PATH:[PATH_OF_POETRY_DIRECTORY]/bin"` to the shell file,
changing `[PATH_OF_FLUTTER_GIT_DIRECTORY]` into the path of your Poetry directory.
- On Windows, From the Start search bar, enter `env` and open `Edit environment variables for your account`.
Under `User variables`, Find (or create) an entry called `Path`
and append the full path to Poetry binary using `;` as a separator from existing values.

Copy link
Member

Choose a reason for hiding this comment

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

I'd expect most developers know how to add something to their path. I understand that their are inexperienced users who don't know. But I'd put this at the end of this section so that the existing information, which I consider more relevant, is still at the top.

docs/_index.md Outdated Show resolved Hide resolved

```bash
curl -sSL https://install.python-poetry.org | python3 - --uninstall
curl -sSL https://install.python-poetry.org | POETRY_UNINSTALL=1 python3 -
Copy link
Member

Choose a reason for hiding this comment

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

Why remove that? It shows one of the options that is described above?

temeddix and others added 3 commits August 26, 2023 18:13
Co-authored-by: Randy Döring <[email protected]>
Co-authored-by: Randy Döring <[email protected]>
Co-authored-by: Randy Döring <[email protected]>
@temeddix temeddix closed this by deleting the head repository Nov 24, 2023
Copy link

github-actions bot commented Mar 3, 2024

This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Mar 3, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
impact/docs Contains or requires documentation changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants