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

update subxt #571

Merged
merged 12 commits into from
Jun 1, 2023
Merged

update subxt #571

merged 12 commits into from
Jun 1, 2023

Conversation

niklasad1
Copy link
Member

No description provided.

src/prelude.rs Outdated Show resolved Hide resolved
@niklasad1 niklasad1 changed the title WIP: update subxt update subxt May 23, 2023
@niklasad1 niklasad1 marked this pull request as ready for review May 23, 2023 15:06
@niklasad1 niklasad1 requested review from jsdw, lexnv and gpestana June 1, 2023 14:13
@@ -37,7 +37,7 @@ once_cell = "1.17"
[dev-dependencies]
anyhow = "1"
assert_cmd = "2.0"
sp-storage = "11.0.0"
sp-storage = { git = "https://github.com/paritytech/substrate" }
Copy link

Choose a reason for hiding this comment

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

nit: Would the latest release 13.0.0 work here?

Copy link
Member Author

Choose a reason for hiding this comment

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

yeah but I changed it because it doesn't make sense to make the dependency tree even bigger :)

Copy link
Member Author

Choose a reason for hiding this comment

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

I need to release a bunch crates for that and my goal is to use master branches here and check out crates.io version on each release.

Because it's so annoying to patch crates if we need some specific from substrate that isn't released.

@niklasad1 niklasad1 merged commit 150e021 into main Jun 1, 2023
@niklasad1 niklasad1 deleted the update-subxt branch June 1, 2023 16: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.

3 participants