Skip to content
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

bpo-34624: Allow regex for module passed via -W or PYTHONWARNINGS #9358

Open
wants to merge 4 commits into
base: master
from

Conversation

Projects
None yet
4 participants
@coldfix
Copy link

commented Sep 17, 2018

Enables passing in regexes for the module part via -W or PYTHONWARNINGS as implied it should be possible by the documentation.

https://bugs.python.org/issue34624

The same should should probably be done for the message part as this is implied to work as regex as well.

https://bugs.python.org/issue34624

@the-knights-who-say-ni

This comment has been minimized.

Copy link

commented Sep 17, 2018

Hello, and thanks for your contribution!

I'm a bot set up to make sure that the project can legally accept your contribution by verifying you have signed the PSF contributor agreement (CLA).

Unfortunately our records indicate you have not signed the CLA. For legal reasons we need you to sign this before we can look at your contribution. Please follow the steps outlined in the CPython devguide to rectify this issue.

You can check yourself to see if the CLA has been received.

Thanks again for your contribution, we look forward to reviewing it!

coldfix added some commits Sep 17, 2018

Allow regex for module via -W or PYTHONWARNINGS
As implied it should be possible by the documentation.
Fix bogus example supposedly matching submodules
The previous expression seems to never have been functional at all.
Fix docs error due to using py3 lexer
Fixes:

    /home/travis/build/python/cpython/Doc/library/warnings.rst:204:
    Could not lex literal_block as "python3". Highlighting skipped.

@coldfix coldfix force-pushed the coldfix:warnings_option_module-regex branch from af66827 to f9b0217 Sep 30, 2018

@kernc

kernc approved these changes Dec 8, 2018

Show resolved Hide resolved Doc/library/warnings.rst Outdated
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.