starlette
starlette copied to clipboard
Skip magic trailing comma
Summary
There have been many comments asking contributors to merge lines where possible such as this one. There is an option available in the Ruff formatter to do this automatically, at the expense of ignoring magic trailing commas: skip-magic-trailing-comma, and a sibling isort option for imports: split-on-trailing-comma.
This change reformats a lot of code, so I would understand if you prefer not merging this.
Checklist
- [x] I understand that this PR may be closed in case there was no previous discussion. (This doesn't apply to typos!)
- [ ] I've added a test for each change that was introduced, and I tried as much as possible to make a single atomic change.
- [ ] I've updated the documentation accordingly.
We can merge this and, add a .git-blame-ignore-revs in a subsequent commit.
Yeah, let's not merge this. 👍
Thanks for letting me know about those flags, anyway.