merge all configs in pyproject.toml

[Borda]

as pyproject.toml already exists in this repo and it is the new python standard, it makes sense to pull all particular configurations and drop redundant setup.cfg.

As Flake8 does not support pyproject.toml I have moved all essential arguments to the single call in CI 🐰

[codecov]

Codecov Report

Patch coverage has no change and project coverage change: -0.10 ⚠️

Comparison is base (2edd849) 94.93% compared to head (9fcf7b2) 94.83%.

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1433      +/-   ##
==========================================
- Coverage   94.93%   94.83%   -0.10%     
==========================================
  Files          44       44              
  Lines        7439     7439              
==========================================
- Hits         7062     7055       -7     
- Misses        377      384       +7     

see 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

[Borda]

@ogrisel @lesteve @tomMoral may I ask you for feedback? 🐰

[Borda]

friendly ping @ogrisel @lesteve @tomMoral 🦩

[tomMoral]

This is a nice PR. We will keep setup.cfg for now to make it easy to run flake8 on the project but this is definitely a step in the right direction. Maybe it would make sense to move to ruff to drop flake8 and thus setup.cfg. PR welcome if you have the time :)

We just duplicated the PR in #1460 to be able to merge this before the release today.

[Borda]

PR welcome if you have the time :)

@tomMoral, I feel confused when you welcome PR, but instead of adjusting mine/existing, you create a new one with another author... that is not a good way how to motivate anyone to contribute more, sorry 😞

Maybe it would make sense to move to ruff to drop flake8 and thus setup.cfg

That was my goal, but as I said not very happy 😕

[Borda]

@fcharras @tomMoral so switch the flake8 t ruff in this PR or in another? doing it here would be trivial, just though to keep a PR minimal and focus

[tomMoral]

LGTM

[Borda]

@tomMoral @fcharras thank you, and apology if I was too straight... 🐰

[Borda]

the failing docs build seems unrelated and appears on multiple PRs

[tomMoral]

Yes this was fixed in #1461 , the PR should now be green for merge