-
-
Notifications
You must be signed in to change notification settings - Fork 32.2k
bpo-39427: Document -X opt options in the CLI --help and the man page #18131
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
Conversation
checks which are too expensive to be enabled by default. It will not be | ||
more verbose than the default if the code is correct: new warnings are | ||
only emitted when an issue is detected. Effect of the developer mode: | ||
* Add default warning filter, as -W default |
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.
I'm not sure if the manual page should elaborate on what are the effects of the development mode. It evolved frequently, and sadly it's common to forget to update the manual page.
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.
I'm not sure if the manual page should elaborate on what are the effects of the development mode. It evolved frequently, and sadly it's common to forget to update the manual page.
I still think is a good idea to have it in the man page but if you feel strongly I can remove it :)
I was just looking at the help to figure out what these were and had to go look at source code, so i think this looks great. I do wonder if there should eventually be two levels of help but this seems on par w/ having the env vars documented in help. |
Thanks @pablogsal for the PR 🌮🎉.. I'm working now to backport this PR to: 3.7, 3.8. |
Sorry @pablogsal, I had trouble checking out the |
Sorry, @pablogsal, I could not cleanly backport this to |
Will backport this tomorrow :) |
…n page (pythonGH-18131) https://bugs.python.org/issue39427 Automerge-Triggered-By: @pablogsal (cherry picked from commit 41f0ef6) Co-authored-by: Pablo Galindo <[email protected]>
GH-18133 is a backport of this pull request to the 3.8 branch. |
GH-18134 is a backport of this pull request to the 3.7 branch. |
…n page (pythonGH-18131) https://bugs.python.org/issue39427 Automerge-Triggered-By: @pablogsal. (cherry picked from commit 41f0ef6) Co-authored-by: Pablo Galindo <[email protected]>
…n page (GH-18131) (GH-18133) https://bugs.python.org/issue39427 Automerge-Triggered-By: @pablogsal (cherry picked from commit 41f0ef6) Co-authored-by: Pablo Galindo <[email protected]>
…n page (GH-18131) (#18134) https://bugs.python.org/issue39427 Automerge-Triggered-By: @pablogsal. (cherry picked from commit 41f0ef6) Co-authored-by: Pablo Galindo <[email protected]>
https://bugs.python.org/issue39427
Automerge-Triggered-By: @pablogsal