[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-08-03 Thread Vinay Sajip
Vinay Sajip added the comment: No, that's OK - leave it as is, I'll consider it when next visiting the area. -- ___ Python tracker ___ ___

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-08-02 Thread Preston Landers
Preston Landers added the comment: A colleague pointed out that I used single quotes in the defaults where the line uses double quotes for another argument. I'm not sure if this is considered a problem but I could submit an update if it is. -- ___ P

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-08-02 Thread Vinay Sajip
Changes by Vinay Sajip : -- resolution: -> fixed stage: -> resolved status: open -> closed ___ Python tracker ___ ___ Python-bugs-li

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-08-02 Thread Vinay Sajip
Vinay Sajip added the comment: New changeset 6ea56d2ebcae69257f8dd7af28c357b25bf394c3 by Vinay Sajip (Preston Landers) in branch 'master': bpo-31080: Allowed logging.config.fileConfig() to accept both args and kwargs. (GH-2979) https://github.com/python/cpython/commit/6ea56d2ebcae69257f8dd7af2

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-08-01 Thread Preston Landers
Changes by Preston Landers : -- pull_requests: +3021 ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://ma

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-08-01 Thread Vinay Sajip
Vinay Sajip added the comment: Yes, seems reasonable, but don't change Misc/NEWS directly in your patch - this is now done using the "blurb" tool. Installed into a Python 3 environment using "pip install blurb". -- ___ Python tracker

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-07-31 Thread Preston Landers
Preston Landers added the comment: This is the current state of my patch: https://github.com/python/cpython/compare/master...Preston-Landers:bpo-31080-fileconfig It makes both `args` and `kwargs` optional in the config file. (Obviously, the actual handler may require args.) -- __

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-07-31 Thread Antoine Pitrou
Changes by Antoine Pitrou : -- nosy: +vinay.sajip ___ Python tracker ___ ___ Python-bugs-list mailing list Unsubscribe: https://mail.

[issue31080] Allow `logging.config.fileConfig` to accept kwargs

2017-07-30 Thread Preston Landers
New submission from Preston Landers: The function `logging.config.fileConfig` accepts `args` but it would be nice if it also accepted `kwargs`. A simple patch seems to do it: diff --git a/Lib/logging/config.py b/Lib/logging/config.py index d692514..4672b48 100644 --- a/