ArduPilot / MethodicConfigurator / 27471331351

94%

push

github

refactor(extract_param_defaults): improve code quality

- Extract _get_parm_value() helper to eliminate too-many-branches in
  extract_parameter_values(); add debug logging for skipped PARM messages
- Fix parse_arguments() parameter type (list[str] instead of Namespace)
  and remove unreachable None check after parse_known_args()
- Narrow contextlib.suppress(Exception) to suppress(OSError) in close_log()
- Use mavutil.mavfile as return/parameter type instead of object
- Add pyright: ignore[reportReturnType] for upstream pymavlink stub deficiency
  (CSVReader not extending mavfile)
- Remove redundant Returns: None from output_params() docstring
- Standardise sort_params() qgcs branch to use dict(sorted()) like other branches
- Add trailing period to NO_DEFAULT_VALUES_MESSAGE

1958 of 2029 branches covered (96.5%)

12461 of 13191 relevant lines covered (94.47%)

2.83 hits per line