adrianmo / go-nmea / 336

100%

push

travis-ci

Implement Tagblocks (#78)

* Added support for TagBlocks
TagBlock is now parsed before any other messages and is a part of BaseSentence

* Added tests for linecount and relative time

* Early exit if no matches in tagblock

* Fixed PR comments

* Renamed parseUint to parseInt64

* Added validation for timestamp according to nmea spec

* changed go mod to fork

* Fix checksum ref after branch update

Signed-off-by: Adrián Moreno <adrian@morenomartinez.com>

* Remove tag string constants

Addresses https://github.com/adrianmo/go-nmea/pull/59#discussion_r312140057

* Pass through timestamp, don't try to normalise

Addresses https://github.com/adrianmo/go-nmea/pull/59#discussion_r312141209

* Clean up tag key/value parsing

Addresses https://github.com/adrianmo/go-nmea/pull/59#discussion_r312142942

* Remove commented code

Addresses https://github.com/adrianmo/go-nmea/pull/59#discussion_r312144656

* Explicitly blank values on error returns

Addresses https://github.com/adrianmo/go-nmea/pull/59#discussion_r312145807

* Revert module name to github.com/adrianmo/go-nmea

* Update readme: note tag blocks feature and code example

* Reference better resource for TAG Blocks info

Addresses https://github.com/adrianmo/go-nmea/pull/78#discussion_r474021879

* Make a malformatted field and error

Addresses https://github.com/adrianmo/go-nmea/pull/78#discussion_r474028018

* Remove blank lines

Adresses https://github.com/adrianmo/go-nmea/pull/78#discussion_r474025814

* Find tag block with strings.SplitN instead of regexp

Addresses https://github.com/adrianmo/go-nmea/pull/78#discussion_r474027494

* Strip tagblocks in parseSentence, remove prefix support

* Add tagblock parsing tests for parseSentence()

* Test parseTagblock directly (without the full sentence)

Co-authored-by: Bjarte Klyve Larsen <bjarteklarsen@gmail.com>
Co-authored-by: bjarte <bjarte@maritimeoptima.com>
Co-authored-by: Adrián... (continued)

70 of 70 new or added lines in 2 files covered. (100.0%)

756 of 756 relevant lines covered (100.0%)

440.11 hits per line