Skip to content

Conversation

@frederikrothenberger
Copy link

@frederikrothenberger frederikrothenberger commented Jun 12, 2024

Motivation

In preparation of moving vc_util to this repository, the dependencies need to be adjusted to be in line with what vc_util currently depends on.

Changes

In preparation of moving vc_util to this repository, the following
changes are made:

  • remove direct dependency on iota fork
    • non-validated JWT parsing is now done using serde_json directly
  • move issuer dependencies to workspace Cargo.toml
  • use consistent versions of ic-cdk and ic-cdk-macros

@frederikrothenberger frederikrothenberger force-pushed the frederik/update-deps branch 10 times, most recently from 5726daf to 19d17ac Compare June 13, 2024 13:09
@frederikrothenberger frederikrothenberger changed the title Restructure rust dependencies and switch to dfinity iota fork chore: Restructure rust dependencies and remove iota fork Jun 13, 2024
@frederikrothenberger frederikrothenberger force-pushed the frederik/update-deps branch 2 times, most recently from c33dfc5 to 2a6cf4b Compare June 13, 2024 13:17
In preparation of moving `vc_util` to this repository, the following
changes are made:
* remove direct dependency on iota fork
  * non-validated JWT parsing is now done using `serde_json` directly
* move issuer dependencies to workspace Cargo.toml
* use consistent versions of `ic-cdk` and `ic-cdk-macros`
Copy link

@przydatek przydatek left a comment

Choose a reason for hiding this comment

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

LGTM, thanks!

@frederikrothenberger frederikrothenberger merged commit c581fe7 into main Jun 17, 2024
@frederikrothenberger frederikrothenberger deleted the frederik/update-deps branch June 17, 2024 12:16
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.

2 participants