nmea icon indicating copy to clipboard operation
nmea copied to clipboard

Adds MWV and MDA parsers

Open AndrewLipscomb opened this issue 3 years ago • 3 comments

Adds a pair of parsers for WIMDA and WIMWV. No GPS Fix related messaging parsing for this set as they are both weather station based messages.

AndrewLipscomb avatar Oct 30 '22 20:10 AndrewLipscomb

Thank you for working on this! I will make sure to review the PR in the coming days.

elpiel avatar Nov 07 '22 07:11 elpiel

Codecov Report

Base: 79.50% // Head: 79.50% // No change to project coverage :thumbsup:

Coverage data is based on head (379d7a2) compared to base (379d7a2). Patch has no changes to coverable lines.

:exclamation: Current head 379d7a2 differs from pull request most recent head a50f483. Consider uploading reports for the commit a50f483 to get more accurate results

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #63   +/-   ##
=======================================
  Coverage   79.50%   79.50%           
=======================================
  Files          22       22           
  Lines         854      854           
=======================================
  Hits          679      679           
  Misses        175      175           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

codecov[bot] avatar Nov 07 '22 07:11 codecov[bot]

Alrighty - still getting the hang of Rust's docs and linting, but cargo fmt and cargo clippy seem like happy bois now. Added the test case too

Edit - forgot the pub members on MDA - fixed now. Give it a whirl Shirl

AndrewLipscomb avatar Nov 26 '22 00:11 AndrewLipscomb