Skip to content

Conversation

@dataflake
Copy link
Member

@dataflake dataflake commented Sep 29, 2025

This PR brings the current ZODB master branch up to date with the standards we are applying across all zopefoundation packages in terms of supported Python versions and package configuration files.

I realize this is a large PR that contains a lot of cosmetic changes, but since no one has touched ZODB in quite a while that kind of bit rot was bound to happen. There is no point in letting it fall back even further.

Copy link
Member

@mauritsvanrees mauritsvanrees 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, thanks!
I tried it in the Plone 6.1 core development buildout on Python 3.13, and that was fine.

I made two small suggestions, but they need not hold back a merge, so I approve.

Copy link
Member

@icemac icemac left a comment

Choose a reason for hiding this comment

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

LGTM scrolling through the changes expecting that nearly all of them are automatically generated.

@dataflake
Copy link
Member Author

The changes are all auto-generated, correct. I used the NOQA instead of making code changes because the code in question was already old, obsolete and unused. I did apply your suggestions, though, thanks!

@dataflake dataflake merged commit b20f31a into master Oct 1, 2025
23 checks passed
@dataflake dataflake deleted the config-with-pure-python-template-b3663390 branch October 1, 2025 07:18
@dataflake
Copy link
Member Author

I have just released ZODB version 6.1.

@navytux
Copy link
Contributor

navytux commented Oct 10, 2025

@dataflake, I'm late to the party but thanks for caring about maintenance.

@dataflake
Copy link
Member Author

@navytux If you have any active branches I would volunteer to merge master.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants