Enable ruff's flake8-commas rule

[CoolCat467]

In this pull request, we enable ruff's flake8-commas rule and handle all the reformatting black does because of this.

No functional changes Had to update gen_exports to run black a 2nd time, if/when this is merged it will probably need to be added to the purely formatting commits thing jakkdl was talking about a while ago.

[codecov]

Codecov Report

Attention: Patch coverage is 25.00000% with 3 lines in your changes missing coverage. Please review.

Project coverage is 99.60%. Comparing base (4ae737a) to head (30babaf).
Report is 7 commits behind head on main.

Files with missing lines	Patch %	Lines
src/trio/_tools/gen_exports.py	25.00%	2 Missing and 1 partial ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3044      +/-   ##
==========================================
- Coverage   99.61%   99.60%   -0.02%     
==========================================
  Files         121      121              
  Lines       17880    17884       +4     
  Branches     3214     3215       +1     
==========================================
+ Hits        17812    17813       +1     
- Misses         48       50       +2     
- Partials       20       21       +1     

Files with missing lines	Coverage Δ
src/trio/_abc.py	100.00% <ø> (ø)
src/trio/_channel.py	100.00% <ø> (ø)
src/trio/_core/_asyncgens.py	100.00% <ø> (ø)
src/trio/_core/_concat_tb.py	100.00% <ø> (ø)
src/trio/_core/_entry_queue.py	100.00% <ø> (ø)
src/trio/_core/_io_epoll.py	100.00% <ø> (ø)
src/trio/_core/_io_kqueue.py	87.20% <ø> (ø)
src/trio/_core/_io_windows.py	98.80% <ø> (ø)
src/trio/_core/_parking_lot.py	100.00% <ø> (ø)
src/trio/_core/_run.py	99.38% <ø> (ø)
... and 71 more

[jakkdl]

In this pull request, we enable ruff's flake8-commas rule and handle all the reformatting black does because of this.

No functional changes Had to update gen_exports to run black a 2nd time, if/when this is merged it will probably need to be added to the purely formatting commits thing jakkdl was talking about a while ago.

#2837

[CoolCat467]

pre-commit.ci autofix

[A5rocks]

Spot-checks fine. I'm not sure how useful this will be, though forcing noqa and type: ignores onto lines with only that argument is actually useful.

Though, can you put this in a blame-ignore file?

[jakkdl]

Spot-checks fine. I'm not sure how useful this will be, though forcing noqa and type: ignores onto lines with only that argument is actually useful.

Though, can you put this in a blame-ignore file?

it's also generally useful for minimizing diffs when only changing a single argument.

And yes, this should be in blame-ignore. If you wanna be strictly correct you can reshuffle the commits so there's one that is pure refactoring, and one that contains the functional changes to gen_exports. This can for example be done with something like (untested, and might depend on the state of your local repo)

git reset origin/main
git stash push --patch src/trio/_tools/gen_exports.py
[interactively only stash the functional changes]
git commit -am [...]
git stash pop
git commit -am [...]

after some brief testing it appears it's possible to add the hash to .git-blame-ignore-revs and not have it change if the PR is merged with a merge commit, in which case you can also do that in the second commit (or in a third).

If you mess up, there's git reflog, or pulling from the remote branch. If you don't manage to work it out lmk and I can do it :)

[jakkdl]

https://github.com/python-trio/trio/blame/main/docs/source/conf.py
whoop whoop!