pipx icon indicating copy to clipboard operation
pipx copied to clipboard

docs: change --global parameter position on ensurepath command

Open marcosalberto opened this issue 1 year ago • 1 comments

…tation

  • [x] I have added a news fragment under changelog.d/ (if the patch affects the end users)

Summary of changes

Change --global parameter position on ensurepath command

Test plan

Tested by running

➜  ~ sudo pipx ensurepath --global
Password:
usage: pipx [-h] [--quiet] [--verbose] [--global] [--version]
            {install,uninject,inject,upgrade,upgrade-all,uninstall,uninstall-all,reinstall,reinstall-all,list,interpreter,run,runpip,ensurepath,environment,completions} ...
pipx: error: unrecognized arguments: --global

➜  ~ sudo pipx --global ensurepath
/usr/local/bin is already in PATH.

⚠️  All pipx binary directories have been added to PATH. If you are sure you want to proceed, try again with the '--force' flag.

Otherwise pipx is ready to go! ✨ 🌟 ✨
➜  ~

marcosalberto avatar Apr 10 '24 15:04 marcosalberto

I believe we could also add the --global option to the shared parser, so that the original command works? This syntax is a bit unpractical.

chrysle avatar Apr 11 '24 05:04 chrysle

I believe we could also add the --global option to the shared parser, so that the original command works?

I am +1 on this.

dukecat0 avatar May 11 '24 13:05 dukecat0

@marcosalberto Thanks for your contribution, but I think it's better to fix the problem from the source, instead of asking the user to change the order.

dukecat0 avatar May 11 '24 13:05 dukecat0

@dukecat0 for sure, my PR aimed to fix the documentation fast to turn the experience for new users easy.

Thanks.

marcosalberto avatar May 11 '24 13:05 marcosalberto