Skip to content

docs: trim the README to current behavior - #98

Merged
jamesarich merged 1 commit into
masterfrom
docs/readme-trim
Oct 29, 2026
Merged

jamesarich merged 1 commit into
masterfrom
docs/readme-trim

Conversation

@jamesarich

@jamesarich jamesarich commented Oct 29, 2026 •

Copy link
Copy Markdown
Collaborator

Checklist

  • PR title specifically describes the change
  • tools/build_all.py (or CI's board matrix) passes - docs-only change, no source touched

Description of Change

A maintainer review of the sibling docs-site PR said "a lot of these changes are way too verbose. We need to be concise or we lose the reader. We also need to get away from documenting old behavior." This applies the same to the README.

Removes comparisons to older releases and repeated "formerly OTAFIX" text, keeping one line near the top. Minimum versions are stated only where a user needs one ("2.6.0 and later"). Boards, BLE names, install steps, UF2 drive, factory erase, DFU screens, troubleshooting, single-button notes and OTA settings are kept. The #how-this-fits-together, #boards-supported, #ble-advertising-names and #recommended-ota-dfu-settings anchors are unchanged; the Contents list is dropped.

README goes from 409 lines / 4877 words to 308 / 3361.

Summary by CodeRabbit

  • Documentation
    • Reorganized and updated bootloader installation, board support, UF2 behavior, DFU routing, troubleshooting, and device-specific guidance.
    • Clarified version 2.6.0 behavior, including SoftDevice mismatch handling, routing without a valid application, inactivity timeouts, fault reporting, and retries.
    • Updated backup, factory erase, display, Android app, and single-button board instructions; removed older-version comparisons and some explanatory details.

Drop release-by-release comparisons and repeated history; state current behavior with minimum versions once.
@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: a7b210b2-09b5-49c6-9d10-5810f8dc4790
Commits

Reviewing files that changed from the base of the PR and between 9296630 and 327ecdb.

Files selected for processing: 1
  • README.md

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Walkthrough

The README is reorganized and revised. It describes installation paths, UF2 and BLE OTA behavior, firmware updates, troubleshooting, and device-specific guidance, including release 2.6.0 details.

Changes: Bootloader documentation

Layer / File(s) Summary
Installation and UF2 entry
README.md
The introduction, board guidance, and installation instructions describe release assets, recovery, and the paths into UF2 or BLE OTA mode.
UF2 operations and updates
README.md
The README documents UF2 file handling, update acceptance and size limits, application backup, factory erase, display behavior, and the Android upgrade workflow.
Troubleshooting and board notes
README.md
The README revises fault and transfer troubleshooting, OTA settings, and guidance for XIAO, RAK4631, and single-button boards.

Priority: Low

Estimated code review effort: 2 (Simple) | ~10 minutes

Change: Other

Suggested reviewers: oltaco

Merge Risk: Minimal · up to 327ec

This documentation-only change has no established merge-blocking risk.

Pre-merge checks: Passed 5
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: shortening the README to document current behavior.
Docstring Coverage Passed No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentat…
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.
Full details: Docstring Coverage

Explanation

No eligible functions found in the changed files. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff with cyclomatic complexity of at least 2. Documentation-only edits and deletions within surviving functions are included; unchanged diff context and fully deleted functions are excluded. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

A rabbit reads the UF2 guide,
Then checks which board can take the ride.
Through cables, screens, and files anew,
The bootloader’s notes are clearer too.
I thump my paws and hop away!

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

@jamesarich
jamesarich added this pull request to the merge queue Oct 29, 2026
Merged via the queue into master with commit 06ef4fd Oct 29, 2026
26 checks passed
@jamesarich
jamesarich deleted the docs/readme-trim branch October 29, 2026 03:05
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