search

AeroRust / nmea

NMEA 0183 - for communication between marine electronics such as echo sounder, sonars, anemometer, gyrocompass, autopilot, GNSS receivers and many other types of instruments. Defined and controlled by the National Marine Electronics Association (NMEA)
https://crates.io/crates/nmea
Other
56 stars 41 forks source link

Feat: APA sentence #123

Closed elpiel closed 4 months ago

elpiel commented 4 months ago

@ANG13T I did my best to merge the different commits from the 2 different branches and PRs that you opened.

I also had to fix 2 issues - 1 with serde and 1 with the Cargo.toml workspace members. Feel free to take a look and raise any issue you find, I'll merge this once all checks pass.

PRs which this PR is based on: #119 #120

codecov[bot] commented 4 months ago

Codecov Report

Attention: Patch coverage is 85.00000% with 9 lines in your changes are missing coverage. Please review.

Project coverage is 77.96%. Comparing base (62763a5) to head (b337b7c).

Files Patch % Lines
src/sentences/apa.rs 85.96% 8 Missing :warning:
src/parser.rs 0.00% 1 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #123 +/- ## ========================================== + Coverage 77.62% 77.96% +0.33% ========================================== Files 34 35 +1 Lines 1247 1307 +60 ========================================== + Hits 968 1019 +51 - Misses 279 288 +9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.