Introduce clippy fixes, auto derive default #60
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
adrian-kong
commented
Sep 6, 2023
gwbres
approved these changes
Sep 6, 2023
aholtzma
pushed a commit
to appliedmind/ublox
that referenced
this pull request
Nov 16, 2023
parser: move test into "mod tests" CI: make sure that we test with all features "on" fix parsing of MonVer packet plus regressioni test for that Fix compilation on `no_std`. Merge pull request ublox-rs#49 from reitermarkus/no-std Fix compilation on `no_std`. Merge branch 'master' into fix-mon-ver-parsing Merge pull request ublox-rs#47 from Dushistov/fix-mon-ver-parsing Fix mon ver parsing Fix enum casing. (ublox-rs#50) Update minimum required version to 1.65 (ublox-rs#58) * Update minimum required version to 1.65 --------- Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com> add support for UBX-SEC-UNIQID (ublox-rs#48) Co-authored-by: gwbres <guillaume.bressaix@gmail.com> introduce cargo fmt and execute it a first time (ublox-rs#59) Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com> Introduce clippy fixes, auto derive default (ublox-rs#60) * apply clippy fixes, derive Default enum * Clippy on CI level * Swap to match statements * Derive eq bump rustc and dependencies to newer versions (ublox-rs#61) * bumps dependencies for ublox library - bumps all dependencies to more recent versions - lints Cargo.toml files - updates ublox_cli example to work with new `clap` and `serialport` APIs - tested ublox_cli example on M8L device - update CI and README with rustc version - fix urls in badges Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> * move cli example to examples - exclude the examples from the top level workspace - allow to build examples independently and with newer MVRC - update CI to build examples separately Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> * fix clippy & fmt errors Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> --------- Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> add ci workflow badge to the readme (ublox-rs#62) Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com> add hnr messages (ublox-rs#63) * rename UBX-NAV-PVT message as per uBlox protocol description Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> * refactor: add more NAV-PVT mapping functions Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> * add HNR-ATT and HNR-INS messages Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> --------- Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> use uniform `itow` naming across packets (ublox-rs#64) Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> release v0.4.3 (ublox-rs#65) Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com> clippy changes (ublox-rs#67) Improve support for embedded targets (ublox-rs#69) * ci: test building on embedded targets * Set resolver=2 in the workspace * ci: test with feature serde but no alloc * derive: Manually impl Serialize for PacketSerializer to avoid the serde feature requiring alloc bump ublox_derive to v0.1.0 (ublox-rs#70) * bump ublox_derive to v0.1.0 * ublox: use local referencing (latest) --------- Signed-off-by: Guillaume W. Bres <guillaume.bressaix@gmail.com> extend basic_cli example with more cmd line arguments (ublox-rs#71) * refactor: workflow file Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> * refactor: small update in doc & README Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> * expand ublox-cli example with more cli-args Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> --------- Signed-off-by: Andrei Gherghescu <8067229+andrei-ng@users.noreply.github.com> Add NAV-SIG and CFG-GNSS messages. Merge remote-tracking branch 'upstream/master' into nav-sig Fix PR
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Some minor clean up