Repository navigation
docs: drop --dma-bisect from README - #14
Conversation
The flag never landed on main; PR #11 was closed once ESPresense#2309 was fixed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
No actionable comments were generated in the recent review.
 |
| Layer / File(s) | Summary |
|---|---|
Update tool description README.md |
The hil_monitor.py description no longer mentions the --dma-bisect mode. |
Possibly related PRs
- ESPresense/firmware-tester#12: Adds the README text that this pull request removes.
Change: Other
Merge Risk: 
ad70a
-
This documentation-only removal presents no identified merge-blocking risk.
Pre-merge checks: 
-
Check name Status Explanation Description Check Check skipped - CodeRabbit’s high-level summary is enabled. Title check The title clearly and concisely describes the main change: removing --dma-bisectfromREADME.md.Docstring Coverage 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 Check skipped because no linked issues were found for this pull request. Out of Scope Changes check 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.)
-
Finishing Touches
-
Generate unit tests (beta)
-
- Commit to this branch
- Create a new PR
-
- 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.
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
The README documents
--dma-bisect, but that flag only existed on thediag/dma-bisectbranch. PR #11 was closed because ESPresense#2309 is fixed, so the flag won't land onmain.🤖 Generated with Claude Code
Summary by CodeRabbit
--dma-bisectas an option for A/B browser-load heap testing. This change affects the documentation only; it does not indicate that the option itself was removed.