Skip to content

Exclude Markdown from ruff format#8940

Merged
robert3005 merged 1 commit into
developfrom
rk/ruff-md-format-exclude
Jul 23, 2026
Merged

Exclude Markdown from ruff format#8940
robert3005 merged 1 commit into
developfrom
rk/ruff-md-format-exclude

Conversation

@robert3005

Copy link
Copy Markdown
Contributor

Rationale for this change

The Python (lint) CI job is failing on develop: CI installs ruff unpinned via uvx, and ruff 0.16 started formatting Python code blocks inside Markdown files by default. Three docs (docs/user-guide/spark.md, java/vortex-spark/README.md, vortex-python-cuda/README.md) use deliberate comment alignment in their examples and now fail ruff format --check.

What changes are included in this PR?

Adds exclude = ["*.md"] under [tool.ruff.format], restoring the pre-0.16 behavior (Python sources only — 137 files — are formatted; ruff check is unaffected). Verified locally with ruff 0.16.0: uvx ruff format --check . and uvx ruff check . both pass.

What APIs are changed? Are there any user-facing changes?

None — lint configuration only.

🤖 Generated with Claude Code

ruff 0.16 (installed unpinned via uvx in CI) started formatting Python
code blocks inside Markdown files by default, breaking the Python lint
job on three docs that use deliberate comment alignment. Keep Markdown
out of the formatter to restore the pre-0.16 behavior; ruff check is
unaffected.

Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
Signed-off-by: Robert Kruszewski <github@robertk.io>
@robert3005 robert3005 added the changelog/chore A trivial change label Jul 23, 2026
@robert3005
robert3005 enabled auto-merge (squash) July 23, 2026 22:48
@robert3005
robert3005 merged commit 2794f89 into develop Jul 23, 2026
82 of 83 checks passed
@robert3005
robert3005 deleted the rk/ruff-md-format-exclude branch July 23, 2026 22:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants