Hi Adam, just FYI small cleanups like this are generally avoided to reduce commit history noise - though Sarah or Nessita may have changed this policy recently and still accept it 👍

Ok, interesting! I don't mind if it gets closed though it could help with linting in the future.

Is the goal of reducing commit history noise documented somewhere? This is just about all I could find about trivial changes, and it seems to encourage them: https://docs.djangoproject.com/en/dev/internals/contributing/writing-code/submitting-patches/#typo-fixes-and-trivial-documentation-changes

Thanks!

Given that this is "in the way" of contributors when proposing PRs with docs changes (see being), I'm very much in favor of merging this. Will do so now!

A side note: these changes cause conflicts in stable/5.1.x so I will not backport them (there is no need to do so).

@nessita do you think it is worth adding the commit for this to

https://github.com/django/django/blob/main/.git-blame-ignore-revs

@nessita do you think it is worth adding the commit for this to

https://github.com/django/django/blob/main/.git-blame-ignore-revs

Thank you @smithdc1 for pointing this out. In all honesty, I did not know this file existed until now 🤯

To answer your inquiry, I've been browsing the git history, and as far as I can see, we haven't been been doing this addition for small formatting changes, see for example 0f694ce or 5a37255. I also consulted with Sarah and we agreed that it would be more suitable to only ignore revnos with larger changes.

Lastly, my personal opinion is that having revnos listed in the git history but not in git blame is a bit unexpected, so I would prefer to stick with the current apparent pattern of ignoring big formatting revisions.