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

Fix: mistake in configuration hints in pyenv init and manpage #3145

Merged
merged 2 commits into from
Dec 23, 2024

Conversation

ChristianFredrikJohnsen
Copy link
Contributor

Description

A silly documentation mistake I made in my previously merged stuff.
Please don't shoot me 😘

-> Currently we get:

pyenv init
# Load pyenv automatically by appending
# the following to
# ~/.zprofile (for login shells)
# and ~/.zshrc (for interactive shells) :

export PYENV_ROOT="$HOME/.pyenv"
[[ -d $PYENV_ROOT/bin ]] && export PATH="$PYENV_ROOT/bin:$PATH"
eval "$(pyenv init -zsh)"

But it should obviously be pyenv init - zsh.

Tests

  • My PR adds the following unit tests (if any)

none

-> Currently we get:
```zsh
pyenv init
# Load pyenv automatically by appending
# the following to
# ~/.zprofile (for login shells)
# and ~/.zshrc (for interactive shells) :

export PYENV_ROOT="$HOME/.pyenv"
[[ -d $PYENV_ROOT/bin ]] && export PATH="$PYENV_ROOT/bin:$PATH"
eval "$(pyenv init -zsh)"
```

But it should obviously be `pyenv init - zsh`.
@native-api native-api merged commit 4c6b0e9 into pyenv:master Dec 23, 2024
18 checks passed
@native-api native-api changed the title fix: silly docs mistake in pyenv init. Fix: mistake in configuration hints in pyenv init and manpage Dec 23, 2024
@ChristianFredrikJohnsen ChristianFredrikJohnsen deleted the fix/pyenv-init-docs branch December 25, 2024 11:50
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