Skip to content

Conversation

@Agent-Hellboy
Copy link
Contributor

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

@StanFromIreland
Copy link
Member

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

Copy link
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

@StanFromIreland
Copy link
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

@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

@python-cla-bot
Copy link

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
Contributor Author

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants