Changed 'python -mpip' to 'python -m pip' for consistency - #19739
Conversation
The top-level files INSTALL.rst and README.rst both use the version with the space between the -m and the module that is execute. To be more consistent, this was changed here as well.
tacaswell
left a comment
tacaswell
left a comment
There was a problem hiding this comment.
I have a weak preference for the other cases to be converted as well, but not going to block on that getting done.
|
I also tend to type with no space (because why bother), but I recently found out that if the module is a dotted name, then powershell gets confused if there's no space (see the point about |
|
Thank you for your contribute @chbauman ! Congratulations on your first (I think) Matplotlib PR 🎉 hopefully we will hear from you again. |
|
Thanks, was actually my second one, but it's also the first project I am contributing to. You will certainly hear more from me. |
|
@chbauman 🐑 sorry for my mistake, but still glad to have you helping! |
…p' for consistency
…739-on-v3.4.x Backport PR #19739 on branch v3.4.x (Changed 'python -mpip' to 'python -m pip' for consistency)
5 participants
The top-level files INSTALL.rst and README.rst both use the version with the space between the -m and the module in the command that is executed. To be more consistent, this was changed here as well.
PR Summary
PR Checklist
pytestpasses).flake8on changed files to check).flake8-docstringsand runflake8 --docstring-convention=all).doc/users/next_whats_new/(follow instructions in README.rst there).doc/api/next_api_changes/(follow instructions in README.rst there).