You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
Repository navigation
Logging: SysLogHandler.createSocket added in 3.11 but not documented #98307
A createSocket() method was added to the SysLogHandler class for issue #88457 by PR #26490 to match the one in SocketHandler, but unlike the latter, the former was not documented (and also not mentioned in What's New).
@vsajip , shouldn't this be documented? If so, do you think you could help draft something describing its semantics and usage (I can help do the final writeup, but its not entirely clear to me what the precise intended behavior is, since it doesn't seem too similar internally to the other documented function, and there is no docstring). Thanks!
Also, I noticed logging.getLevelNamesMapping was added as well, but not mentioned in What's New. Seems like at least a brief note/reference would be worth adding, too (which I could easily do on my own as part of or separate from this, as its already documented).
Also, I noticed logging.getLevelNamesMapping was added as welll, but not mentioned in What's New ... which I could easily do on my own as part of or separate from this
I'd say please do it separately, as it's unrelated to the socket creation part of the issue.
Sure; the main reason I didn't just do it separately to begin with was the PRs would conflict (at least as far as the What's New is concerned), but I just added the two What's New entries to the same PR, leaving your PR to document createSocket as a separate one (which makes a lot more sense anyway).
Activity
CAM-Gerlach commentedon Oct 16, 2022
Also, I noticed logging.getLevelNamesMapping was added as well, but not mentioned in What's New. Seems like at least a brief note/reference would be worth adding, too (which I could easily do on my own as part of or separate from this, as its already documented).
vsajip commentedon Oct 16, 2022
I'd say please do it separately, as it's unrelated to the socket creation part of the issue.
pythongh-98307: Add docstring and documentation for SysLogHandler.cre…
CAM-Gerlach commentedon Oct 16, 2022
Thanks for the quick response!
Sure; the main reason I didn't just do it separately to begin with was the PRs would conflict (at least as far as the What's New is concerned), but I just added the two What's New entries to the same PR, leaving your PR to document
createSocketas a separate one (which makes a lot more sense anyway).See #98320 for that.
gh-98307: Add docstring and documentation for SysLogHandler.createSoc…
[3.11] pythongh-98307: Add docstring and documentation for SysLogHand…
[3.11] gh-98307: Add docstring and documentation for SysLogHandler.cr… (
Merge branch 'main' into typewatch