Skip to content

tooling: skip magic trailing commas in ruff format - #70

Merged
jmccarrell merged 1 commit into
mainfrom
18-21-tooling
May 6, 2026
Merged

tooling: skip magic trailing commas in ruff format#70
jmccarrell merged 1 commit into
mainfrom
18-21-tooling

Conversation

@jmccarrell

Copy link
Copy Markdown
Owner

Summary

  • Add `format.skip-magic-trailing-comma = true` and `lint.isort.split-on-trailing-comma = false` to ruff.toml so the formatter respects existing layout choices and only splits when the unsplit form would exceed `line-length = 108`.
  • Apply the resulting formatter sweep to `docs/conf.py`, `init.py`, `changes.py`, `n100tickers.py`, and `test_changes.py` (compaction of previously-split function signatures and collection literals).

Test plan

  • `just lint` exits 0 on a clean tree
  • `just lint` modifies no files (idempotent)
  • CI green on this PR

🤖 Generated with Claude Code

Add `format.skip-magic-trailing-comma = true` and matching
`lint.isort.split-on-trailing-comma = false` to ruff.toml so the
formatter respects existing layout choices and only splits when the
unsplit form would exceed `line-length = 108`.

Apply the resulting formatter sweep to docs/conf.py, __init__.py,
changes.py, n100tickers.py, and test_changes.py.
@jmccarrell
jmccarrell merged commit df9ee3e into main May 6, 2026
2 checks passed
@jmccarrell
jmccarrell deleted the 18-21-tooling branch May 6, 2026 15:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant