Skip to content

Conversation

adrian-kong
Copy link
Contributor

@adrian-kong adrian-kong commented Nov 30, 2022

  • Moving some bench / test dependencies into [dev-dependencies] reduce # of dependencies pulled
  • Reorder all optional feature in cargo.toml
  • Remove EnumVariant to use ValueEnum from claps
  • Update to clap 4
  • Update all deps

@adrian-kong adrian-kong changed the title Refactor dependencies Refactor dependencies, clap 4 Nov 30, 2022
@adrian-kong adrian-kong changed the title Refactor dependencies, clap 4 Refactor dependencies, clap 4 [CPP-841] Nov 30, 2022
@adrian-kong adrian-kong requested a review from a team December 1, 2022 00:31

[build-dependencies]
capnpc = "0.14"
capnpc = "0.15.0"
Copy link
Collaborator

Choose a reason for hiding this comment

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

I know this is modifying only the Rust side of things but we did have to fix the pycapnp version due to a bug that popped up. This may not event be an issue in newer versions if any:
https://swift-nav.atlassian.net/browse/CPP-763

@adrian-kong
Copy link
Contributor Author

adrian-kong commented Dec 2, 2022

I have no clue what is going on with cargo clippy...
It works fine locally, seems out of date - which is odd ?
🤔

Edit: think its fixed
Edit edit: nvm guess will have to revert to using strum

EDIT EDIT EDIT: nvm its the pyrs transpiler

@adrian-kong adrian-kong enabled auto-merge (squash) December 5, 2022 02:02
@adrian-kong adrian-kong disabled auto-merge December 5, 2022 04:09
@swiftnav-travis
Copy link

Frontend and Release Workflow Started here

@adrian-kong adrian-kong merged commit 36d7a17 into main Dec 5, 2022
@adrian-kong adrian-kong deleted the adrian/deps branch December 5, 2022 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants