Skip to content

Use spaces instead of tabs in markdown files - #56

Merged
snelusha merged 1 commit into
mainfrom
refactor/docs
Aug 9, 2026
Merged

Use spaces instead of tabs in markdown files#56
snelusha merged 1 commit into
mainfrom
refactor/docs

Conversation

@snelusha

@snelusha snelusha commented Aug 9, 2026

Copy link
Copy Markdown
Owner

Summary by CodeRabbit

  • Documentation

    • Updated README code examples with consistent two-space indentation across React, stream writer, filesystem, and HTTP service examples.
  • Style

    • Added Markdown formatting guidance to use spaces instead of tabs while preserving existing ignore patterns.

@coderabbitai

coderabbitai Bot commented Aug 9, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 6979c616-4f62-47c1-834b-23be7f518ab3

📥 Commits

Reviewing files that changed from the base of the PR and between 077b448 and 4fe0c05.

📒 Files selected for processing (2)
  • .oxfmtrc.json
  • packages/balrun/README.md

📝 Walkthrough

Walkthrough

The PR configures Oxfmt to disable tabs in Markdown files. It also reformats Balrun README code examples with two-space indentation without changing their behavior.

Changes

Markdown formatting alignment

Layer / File(s) Summary
Markdown formatter override
.oxfmtrc.json
Adds a Markdown-specific useTabs: false override and retains the existing ignore patterns.
README example indentation
packages/balrun/README.md
Reformats React, stream writer, filesystem, and HTTP examples with two-space indentation.

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

Possibly related PRs

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title clearly and concisely describes the main change: replacing tabs with spaces in Markdown files.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch refactor/docs

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.

@snelusha
snelusha merged commit 50870e2 into main Aug 9, 2026
2 checks passed
@snelusha
snelusha deleted the refactor/docs branch August 9, 2026 17:39
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