NotificationsYou must be signed in to change notification settings
Fork3.1k
Star41.4k

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Jump to bottom

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

Merged

native-api merged 2 commits intopyenv:masterfromChristianFredrikJohnsen:fix/pyenv-init-docs

Dec 23, 2024

Merged

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

native-api merged 2 commits intopyenv:masterfromChristianFredrikJohnsen:fix/pyenv-init-docs

Dec 23, 2024

Conversation

Copy link

Contributor

ChristianFredrikJohnsen commentedDec 22, 2024

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 bepyenv init - zsh.

Tests

My PR adds the following unit tests (if any)

none

ChristianFredrikJohnsenand others added2 commits

December 22, 2024 21:55

fix: fix silly docs mistake inpyenv init.

df4c4cd

-> Currently we get:```zshpyenv 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`.