Skip to content

Switch to pyproject.toml based project setup#69

Merged
henri-hulski merged 1 commit into
morepath:masterfrom
seantis:chore/pyproject.toml
May 28, 2026
Merged

Switch to pyproject.toml based project setup#69
henri-hulski merged 1 commit into
morepath:masterfrom
seantis:chore/pyproject.toml

Conversation

@Daverball
Copy link
Copy Markdown
Contributor

@Daverball Daverball commented May 27, 2026

Based on the equivalent changes to morepath made by @henri-hulski .

I did however decide to add flake8-pyproject to the development dependencies, so we can put the flake8 configuration inside pyproject.toml as well.

Long term it might make more sense to replace black and flake8 with ruff.

I also decided to use tox-gh instead of creating a separate runner for every tox environment, since that seems a bit excessive, considering how fast most checks are.

This is preparation for adding inline type hints to reg, I wanted a more modern project configuration, before I start adding type checkers to it.

@henri-hulski
Copy link
Copy Markdown
Member

henri-hulski commented May 27, 2026

Seems to be a problem with coverall. I think we should remove it as we have 100% coverage.

If you want to switch to rust feel free to go. I'm also starting to switch my projects. I quite like it.

@Daverball
Copy link
Copy Markdown
Contributor Author

Seems to be a problem with coverall. I think we should remove it as we have 100% coverage.

I think it was just a temporary network failure, the same job succeeded on the forked repository. Feel free to re-run the failed jobs.

If you want to switch to rust feel free to go. I'm also starting to switch my projects. I quite like it.

I'm happy to do that in a follow-up PR, for now this is good enough.

@henri-hulski henri-hulski merged commit f500b11 into morepath:master May 28, 2026
11 of 13 checks passed
@Daverball Daverball deleted the chore/pyproject.toml branch May 28, 2026 20:04
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.

2 participants