Skip to content

bump substrate to polkadot-v0.9.2. + add GA + re-add branch ids #116

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

Merged
merged 23 commits into from
May 27, 2021

Conversation

clangenb
Copy link
Collaborator

@clangenb clangenb commented May 11, 2021

Rationale Branch Ids:

Polkadot references substrate dependencies including branch = "master". Hence, any rust crate that uses substrate AND Polkadot deps must use the branch-id because Cargo can't identify them as the same even if the same substrate revision is referenced.

This is the api-client branch that is currently used with the encointer-client.

@clangenb clangenb requested a review from brenzi May 11, 2021 08:27
Copy link
Contributor

@brenzi brenzi left a comment

Choose a reason for hiding this comment

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

still some clippy trouble....

@clangenb
Copy link
Collaborator Author

yes, wasm-toochail was not installed for the clippy runner.

Anyhow. Please don't merge yet. It currently depends on the encointer fork of the substrate-api-client-test-node just to test if things work.

@clangenb clangenb changed the title Reintroduce substrate branch-id bump substrate to polkadot-v0.9.2. May 25, 2021
@clangenb clangenb marked this pull request as draft May 25, 2021 05:51
@clangenb clangenb changed the title bump substrate to polkadot-v0.9.2. bump substrate to polkadot-v0.9.2. + add GA + re-add branch ids May 25, 2021
@clangenb clangenb marked this pull request as ready for review May 25, 2021 14:59
@brenzi brenzi merged commit 3c8c04c into master May 27, 2021
@clangenb clangenb deleted the reintroduce-substrate-branch-id branch May 28, 2021 11:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

bump substrate to polkadot-v0.9.2
3 participants