-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Refactor parsers #51
Refactor parsers #51
Conversation
6919231
to
292f631
Compare
- add GBS to ParseResult - handle BOD & GBS in parser::Nmea
- rename NmeaError to Error and move to module `error` - parse_str & parse_bytes - all_supported_messages test
- add documentation to sentences data structs - change exports of sentences and other types
292f631
to
0ef9271
Compare
Codecov Report
@@ Coverage Diff @@
## main #51 +/- ##
==========================================
+ Coverage 82.64% 85.56% +2.91%
==========================================
Files 15 23 +8
Lines 1268 1247 -21
==========================================
+ Hits 1048 1067 +19
+ Misses 220 180 -40
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
@Dushistov would you like to take a look at the changes? |
&str
instead of&[u8]
parse_str
&parse_bytes
Support for 2 new message types: