sphinx-doc / sphinx

The Sphinx documentation generator
https://www.sphinx-doc.org/
Other
6.61k stars 2.13k forks source link

Apply assorted ruff rules #13093

Closed DimitriPapadopoulos closed 1 week ago

DimitriPapadopoulos commented 3 weeks ago

Subject: apply new ruff rules

Feature or Bugfix

AA-Turner commented 3 weeks ago

PRs which apply multiple rules at once make history harder to decipher, perhaps we could split to one PR per rule, or per group of rules.

A

DimitriPapadopoulos commented 3 weeks ago

Alternatively, merge without squashing. The PR is already split into multiple commits.

AA-Turner commented 3 weeks ago

Merge commits are disabled for this repo -- there's no option to select it. This is to preserve linear history. (I do appreciate the effort to split into individual commits, though, thank you)

DimitriPapadopoulos commented 1 week ago

Superseded by #13115, #13116, #13117, #13118.