docs(hardware-overview): correct stale DroneCAN claim for INAV 10.0 - #16
Open
daijoubu wants to merge 1 commit into
Open
docs(hardware-overview): correct stale DroneCAN claim for INAV 10.0#16daijoubu wants to merge 1 commit into
daijoubu wants to merge 1 commit into
Conversation
The CAN section in the getting-started hardware overview still says "INAV does not support any DroneCAN based sensors yet", which is true for 9.1.0 but not for 10.0. DroneCAN battery monitor and GNSS receiver support ship in 10.0 (drivers in maintenance-10.x, param-getset firmware PR #11683 already merged); the unversioned docs/ tree describes 10.0. The 9.1.0 versioned snapshot keeps the original wording — it was true at the time of that release and is frozen per the project's versioning policy.
PR Summary by QodoDocument DroneCAN sensor support in INAV 10.0 hardware overview
AI Description
High-Level Assessment
Files changed (1)
|
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can add REVIEW.md to your repo root and Qodo follows it on every PR |
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
The CAN section in the getting-started hardware overview still says "INAV does not support any DroneCAN based sensors yet". That is true for 9.1.0 but not for 10.0: DroneCAN support for battery monitors and GNSS receivers ships in 10.0 (the underlying drivers are in
maintenance-10.x; related firmware PR iNavFlight/inav#11683 is already merged). The unversioneddocs/tree describes 10.0, so the claim is wrong on this page.The 9.1.0 versioned snapshot keeps the original wording — it was true at the time of that release and is frozen per the project's versioning policy.
Changes
docs/03-getting-started/01-hardware-overview.mdx: replace the "INAV does not support any DroneCAN based sensors yet" sentence in the CAN section with a 10.0-era description of the new DroneCAN support.Verification
npm run buildsucceeds with the change applied. Pre-existing</p>HTML-minifier warnings in the same file are unrelated to this change (same count before and after).