Skip to content

Maurits py3 only code quality no pkg resources - #222

Merged
mauritsvanrees merged 7 commits into
maurits-py3-only-code-qualityfrom
maurits-py3-only-code-quality-no-pkg-resources
Aug 20, 2026
Merged

Maurits py3 only code quality no pkg resources#222
mauritsvanrees merged 7 commits into
maurits-py3-only-code-qualityfrom
maurits-py3-only-code-quality-no-pkg-resources

Conversation

@mauritsvanrees

Copy link
Copy Markdown
Collaborator

Fixes issue #218.

Got rid of code that tried to support `zc.buildout` 1.4.
Move the remaining requirements inline, instead of defining them above the setup call.
This is instead of an ugly error:

```
$ bin/develop
Traceback (most recent call last):
  File "/Users/maurits/community/plone-coredev/6.1/bin/develop", line 15, in <module>
    sys.exit(mr.developer.develop.develop())
  File "/Users/maurits/shared-eggs/cp310/mr.developer-2.0.4-py3.10.egg/mr/developer/develop.py", line 103, in __call__
    args.func(args)
AttributeError: 'Namespace' object has no attribute 'func'
```
@mauritsvanrees

Copy link
Copy Markdown
Collaborator Author

This is ready for review. It is part of a stack of three PRs that build on each other, but can be merged and reviewed separately if wanted, starting with the first. Or just merge the current one and all others will be merged automatically in the right order.

  1. PR Python 3 only, require Python 3.10+, support until 3.15 #219
  2. PR Improve code quality and add checkers #220
  3. PR Maurits py3 only code quality no pkg resources #222

The first two contain lots and lots of code changes, but mostly automatic with pyupgrade, isort, black. The last one (the current) is the most interesting, as this fixes compatibility with zc.buildout 6.0.0a1 and setuptools 82+. The others are necessary stepping stones, especially to get to Python 3.10+.

You can review per commit and skip the giant automatic ones. Or just try it out.

Pinging @mamico and @janjaapdriessen for review, as you have been active in this repo lately, with your improvements landing in 2.1.0 which I have released today.

cc @gforcada @icemac @dataflake as they may also want to have a look (also at the released 2.1.0 changes).

I have asked for review from original author @fschulze, but I have not seen him active in this repo for a long time, so he may no longer be using it.

@mauritsvanrees
mauritsvanrees merged commit f918d2c into master Aug 20, 2026
7 checks passed
@mauritsvanrees

Copy link
Copy Markdown
Collaborator Author

Thanks for the reviews!
I have released this in 3.0.0.

@fschulze

Copy link
Copy Markdown
Owner

@mauritsvanrees I'm still using it, but I had no issues and you are doing a fantastic job!

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.

4 participants