FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Format the README JIT example for ruff-format by youknowone · Pull Request #8785 · RustPython/RustPython · GitHub

  1. RustPython
  2. RustPython

Repository navigation

Format the README JIT example for ruff-format - #8785

Merged
youknowone merged 1 commit into
RustPython:mainfrom
youknowone:fix/ruff-format-python-only
Sep 22, 2026
ConversationCommits (1)ChecksFiles changed
Merged

Format the README JIT example for ruff-format#8785
youknowone merged 1 commit into
RustPython:mainfrom
youknowone:fix/ruff-format-python-only

Conversation

youknowone commented Sep 22, 2026 •
edited by coderabbitai Bot
Loading

Copy link
Copy Markdown
Member

One of checkbox below must be checked.

  • I did not use AI to write the code of this patch.
  • This PR follows our AI policy

Summary

#8777 fails Lint because ruff-format 0.16.7 now formats Python code blocks in Markdown. The JIT snippet in README.md had one blank line after a top-level def; the formatter wants two.

Format that example so the bump can land. Do not pin the hook away from Markdown.

Assisted-by: Grok:4.6

Summary by CodeRabbit

  • Documentation
    • Fixed trailing whitespace in a JIT compiler documentation code example.

chatgpt-codex-connector Bot commented Sep 22, 2026

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

coderabbitai Bot commented Sep 22, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info ⚙️ Run configuration

Configuration used: Repository: RustPython/RustPython/.coderabbit.yml

Review profile: CHILL

Plan: Advanced

Run ID: adae77bf-d237-426c-88c0-17d8cf2ffb64

📥 Commits

Reviewing files that changed from the base of the PR and between 59a71a4 and 24038a2.

📒 Files selected for processing (1)
  • README.md

Included review availability: Your plan provides up to 10 included reviews per hour; 6 remain after this review.

📝 Walkthrough

Walkthrough

The README removes trailing whitespace from a blank line in the JIT compiler __jit__() documentation example. No functional or textual content changes.

Changes

README whitespace cleanup

Layer / File(s) Summary
Clean JIT example whitespace
README.md		 The blank line in the __jit__() documentation example no longer contains trailing whitespace.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Merge Risk: ⚪ Minimal · up to 24038

This documentation-only formatting change has no runtime or user-facing behavior impact and is ready to merge.

🚥 Pre-merge checks | ✅ 5 ✅ Passed checks (5 passed)
Check name Status Explanation
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: formatting the README JIT example for ruff-format.
✨ Finishing Touches 🧪 Generate unit tests (beta)
  • Create a new PR

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

youknowone changed the title Pin ruff-format to Python files only Format the README JIT example for ruff-format Sep 22, 2026
Format the README JIT example for ruff-format
24038a2 
ruff-format 0.16.7 formats Python fences in Markdown and wants two
blank lines after a top-level function.

Assisted-by: Grok:4.6
youknowone force-pushed the fix/ruff-format-python-only branch from be4c725 to 24038a2 Compare September 22, 2026 01:47
youknowone merged commit 23fcb8d into RustPython:main Sep 22, 2026
8 checks passed
youknowone deleted the fix/ruff-format-python-only branch September 22, 2026 01:49
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

Back | FazBrowse Home | New Git URL