Skip to content

Update logging.config.listen() docstrings to mention dictConfig() - #155479

Merged
vsajip merged 1 commit into
python:mainfrom
lpyu001:logging-config-docstring
Aug 10, 2026
Merged

Update logging.config.listen() docstrings to mention dictConfig()#155479
vsajip merged 1 commit into
python:mainfrom
lpyu001:logging-config-docstring

Conversation

@lpyu001

@lpyu001 lpyu001 commented Aug 10, 2026

Copy link
Copy Markdown
Contributor

The logging.config.listen() docstring only mentions fileConfig(),but the implementation first attempts to process JSON configurations using dictConfig() and falls back to fileConfig().

Update the public docstring and the two related local docstrings to describe both supported configuration mechanisms.

@lpyu001
lpyu001 requested a review from vsajip as a code owner August 10, 2026 10:16
@aisk aisk added the skip news label Aug 10, 2026
@aisk aisk changed the title logging.config.listen() docstring omits dictConfig() support Update logging.config.listen() docstrings to mention dictConfig() Aug 10, 2026
@vsajip
vsajip merged commit 791b573 into python:main Aug 10, 2026
63 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in Docs PRs Aug 10, 2026
@lpyu001

lpyu001 commented Aug 11, 2026

Copy link
Copy Markdown
Contributor Author

This also affects the 3.13, 3.14, and 3.15 branches. Should this be backported to these branches as well? @vsajip

@aisk aisk added needs backport to 3.13 bugs and security fixes needs backport to 3.14 bugs and security fixes needs backport to 3.15 pre-release feature fixes, bugs and security fixes labels Aug 11, 2026
@miss-islington-app

Copy link
Copy Markdown

Thanks @lpyu001 for the PR, and @vsajip for merging it 🌮🎉.. I'm working now to backport this PR to: 3.13.
🐍🍒⛏🤖

@miss-islington-app

Copy link
Copy Markdown

Thanks @lpyu001 for the PR, and @vsajip for merging it 🌮🎉.. I'm working now to backport this PR to: 3.14.
🐍🍒⛏🤖

@miss-islington-app

Copy link
Copy Markdown

Thanks @lpyu001 for the PR, and @vsajip for merging it 🌮🎉.. I'm working now to backport this PR to: 3.15.
🐍🍒⛏🤖

@bedevere-app

bedevere-app Bot commented Aug 11, 2026

Copy link
Copy Markdown

GH-155528 is a backport of this pull request to the 3.13 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.13 bugs and security fixes label Aug 11, 2026
@bedevere-app

bedevere-app Bot commented Aug 11, 2026

Copy link
Copy Markdown

GH-155529 is a backport of this pull request to the 3.14 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.14 bugs and security fixes label Aug 11, 2026
@bedevere-app

bedevere-app Bot commented Aug 11, 2026

Copy link
Copy Markdown

GH-155530 is a backport of this pull request to the 3.15 branch.

@bedevere-app bedevere-app Bot removed the needs backport to 3.15 pre-release feature fixes, bugs and security fixes label Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docs Documentation in the Doc dir skip issue skip news

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants