Skip to content
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

bump ublox_derive to v0.1.0 #70

Merged
merged 5 commits into from
Sep 28, 2023
Merged

bump ublox_derive to v0.1.0 #70

merged 5 commits into from
Sep 28, 2023

Conversation

gwbres
Copy link
Collaborator

@gwbres gwbres commented Sep 27, 2023

Following @mattico identification of the problem

Signed-off-by: Guillaume W. Bres <[email protected]>
@gwbres gwbres added the enhancement New feature or request label Sep 27, 2023
@gwbres gwbres requested a review from adrian-kong September 27, 2023 07:56
@gwbres gwbres self-assigned this Sep 27, 2023
@gwbres gwbres linked an issue Sep 27, 2023 that may be closed by this pull request
Signed-off-by: Guillaume W. Bres <[email protected]>
Signed-off-by: Guillaume W. Bres <[email protected]>
@gwbres gwbres marked this pull request as ready for review September 27, 2023 09:22
ublox/Cargo.toml Show resolved Hide resolved
ublox_derive/Cargo.toml Show resolved Hide resolved
@mattico
Copy link
Contributor

mattico commented Sep 27, 2023

I'll also mention for consideration that there are some tools to help with automating releases like this. I know of:

In general they take care of bumping crate versions (and intra-crate dependencies), making git tags, updating changelogs, etc.

For my crates I couldn't be bothered to choose one and figure out how to use it. It didn't seem like that much work to make a release manually. But then again I make a mistake at least 1/5 of releases, so I probably should.

I'm definitely not saying this needs to be implemented and not in this PR, I just wanted to mention it.

Copy link
Collaborator

@adrian-kong adrian-kong left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, just need to release this to resolve the issue

@adrian-kong
Copy link
Collaborator

I'll also mention for consideration that there are some tools to help with automating releases like this. I know of:

I can vouch for cargo release is pretty useful. Although I also prefer releasing manually as how easy it is.

@gwbres
Copy link
Collaborator Author

gwbres commented Sep 28, 2023

Hello @mattico, @adrian-kong

I just published them, see you

@gwbres gwbres merged commit 4e88740 into master Sep 28, 2023
12 checks passed
@mattico
Copy link
Contributor

mattico commented Sep 28, 2023

Thanks @gwbres!

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 <[email protected]>
add support for UBX-SEC-UNIQID (ublox-rs#48)

Co-authored-by: gwbres <[email protected]>
introduce cargo fmt and execute it a first time (ublox-rs#59)

Signed-off-by: Guillaume W. Bres <[email protected]>
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 <[email protected]>

* 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 <[email protected]>

* fix clippy & fmt errors

Signed-off-by: Andrei Gherghescu <[email protected]>

---------

Signed-off-by: Andrei Gherghescu <[email protected]>
add ci workflow badge to the readme (ublox-rs#62)

Signed-off-by: Guillaume W. Bres <[email protected]>
add hnr messages (ublox-rs#63)

* rename UBX-NAV-PVT message as per uBlox protocol description

Signed-off-by: Andrei Gherghescu <[email protected]>

* refactor: add more NAV-PVT mapping functions

Signed-off-by: Andrei Gherghescu <[email protected]>

* add HNR-ATT and HNR-INS messages

Signed-off-by: Andrei Gherghescu <[email protected]>

---------

Signed-off-by: Andrei Gherghescu <[email protected]>
use uniform `itow` naming across packets (ublox-rs#64)

Signed-off-by: Andrei Gherghescu <[email protected]>
release v0.4.3 (ublox-rs#65)

Signed-off-by: Guillaume W. Bres <[email protected]>
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 <[email protected]>
extend basic_cli example with more cmd line arguments (ublox-rs#71)

* refactor: workflow file

Signed-off-by: Andrei Gherghescu <[email protected]>

* refactor: small update in doc & README

Signed-off-by: Andrei Gherghescu <[email protected]>

* expand ublox-cli example with more cli-args

Signed-off-by: Andrei Gherghescu <[email protected]>

---------

Signed-off-by: Andrei Gherghescu <[email protected]>
Add NAV-SIG and CFG-GNSS messages.

Merge remote-tracking branch 'upstream/master' into nav-sig

Fix PR
@gwbres gwbres deleted the next branch December 1, 2023 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Release v0.4.3 needs a new release of ublox_derive
3 participants