Skip to content

gh-127011: Add __str__ and __repr__ to ConfigParser#132966

Open
Agent-Hellboy wants to merge 2 commits intopython:mainfrom
Agent-Hellboy:new-fix-issue-127011
Open

gh-127011: Add __str__ and __repr__ to ConfigParser#132966
Agent-Hellboy wants to merge 2 commits intopython:mainfrom
Agent-Hellboy:new-fix-issue-127011

Conversation

@Agent-Hellboy
Copy link
Copy Markdown
Contributor

@Agent-Hellboy Agent-Hellboy commented Apr 25, 2025

Comment thread Lib/configparser.py
Comment thread Lib/configparser.py
Comment thread Lib/configparser.py
Comment thread Misc/ACKS
Comment thread Lib/test/test_configparser.py
Comment thread Lib/test/test_configparser.py
Comment thread Lib/test/test_configparser.py Outdated
Comment thread Lib/configparser.py Outdated
Comment thread Lib/configparser.py Outdated
Comment thread Lib/configparser.py Outdated
Comment thread Lib/configparser.py
Comment thread Misc/NEWS.d/next/Library/2025-04-25-18-29-10.gh-issue-127011.Ipem5z.rst Outdated
@StanFromIreland
Copy link
Copy Markdown
Member

You can batch apply suggestions from the Files Changed tab, please do so.

Comment thread Misc/NEWS.d/next/Library/2025-04-25-18-29-10.gh-issue-127011.Ipem5z.rst Outdated
Comment thread Lib/configparser.py Outdated
Comment thread Lib/configparser.py Outdated
Comment thread Lib/test/test_configparser.py
Comment thread Lib/test/test_configparser.py Outdated
Copy link
Copy Markdown
Member

@picnixz picnixz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good but I'll let @jaraco have the final say

Comment thread Lib/test/test_configparser.py
Comment thread Lib/configparser.py Outdated
Comment thread Lib/test/test_configparser.py Outdated
Comment thread Lib/test/test_configparser.py Outdated
@StanFromIreland
Copy link
Copy Markdown
Member

You do not need to click the button to update the branch, if it’s needed a core dev will run it before merging.

Copy link
Copy Markdown

@auvipy auvipy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good so far

Comment thread Lib/configparser.py Outdated
Comment thread Lib/configparser.py Outdated
@python-cla-bot
Copy link
Copy Markdown

python-cla-bot Bot commented Aug 19, 2025

All commit authors signed the Contributor License Agreement.

CLA signed

- Implement __str__ method showing formatted configuration content
- Implement __repr__ method with constructor parameters and state summary
- Add comprehensive tests for both methods
- Update ACKS and add NEWS entry
@Agent-Hellboy
Copy link
Copy Markdown
Contributor Author

Hi @jaraco, please merge if everything is done here.

@merwok
Copy link
Copy Markdown
Member

merwok commented Nov 27, 2025

There isn’t a consensus on the methods yet.

(PR discussion focus on the implementation (how to do it), issue discussion on the goals (should we do it / what should we do))

@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open for 30 days with no activity.

@github-actions github-actions Bot added the stale Stale PR or inactive for long period of time. label Apr 23, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DO-NOT-MERGE stale Stale PR or inactive for long period of time.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants