-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
Improve docs for umask option of pip module. #49146
Conversation
Note that the umask should be specified as an octal *string*, not an octal (or decimal) *integer*. +label: docsite_pr
Hi @pillarsdotnet, thank you for submitting this pull-request! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@pillarsdotnet thank you!
Thanks @pillarsdotnet - can you open a backport PR to |
I don't think we want to do this quite like this. We do not want to tell people to use a decimal number in a mode or umask. There's a couple options:
At the very least we need to remove lines 104-106 as we do not document buggy behaviour. |
@abadger maybe we could have some special type and a helper class in basic? It'd unify mask processing across all modules. |
* Improve docs for umask option of pip module; umask should be specified as an octal *string*, not an octal (or decimal) *integer*.
Note that the umask should be specified as an octal string, not an octal (or decimal) integer.
+label: docsite_pr
SUMMARY
Fixes #43256
ISSUE TYPE
COMPONENT NAME
pip