wboayue / stable-flight / 40133612122

91%

Build Type

push

github

Committed by web-flow

Commit Message

Review follow-up for #451: acceptance accessor, doc wraps, X-Plane note (#501)

Follow-up to #500's post-merge review (#451). No behavior change.

- `ScriptCommander`: the per-regime altitude acceptance is read through
a `&self` accessor, so `active_altitude_acceptance_m` no longer needs
`&mut self`; `fly()` reads the active regime once and uses it for both
the acceptance and the wait.
- Rustdoc: reflow two lines left short mid-sentence (`WaypointFollower`,
`ScriptCommander::next_command`).
- AGENTS.md: the X-Plane note no longer says a missed plane leg flies on
past its waypoint. A plane leg passed off altitude now loiters on it
until at altitude, which can add minutes to a lap (the Alia waited 193
s), so watchdog timeouts need that margin.

Tested: `cargo fmt --all`, `just lint`, `cargo test --workspace
--all-features`. No flight: the commands flown are unchanged (the
existing `ScriptCommander` acceptance and back-transition tests pass as
before).

Coverage Stats

12 of 12 new or added lines in 1 file covered. (100.0%)

67 existing lines in 11 files now uncovered.

10413 of 11147 relevant lines covered (93.42%)

8.91 hits per line