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

Polkadot v0.9.29 #282

Closed
wants to merge 7 commits into from
Closed

Polkadot v0.9.29 #282

wants to merge 7 commits into from

Conversation

clangenb
Copy link
Collaborator

patch ed25519-zebra for no_std compatibility. See: paritytech/substrate#12225

@clangenb clangenb marked this pull request as draft September 29, 2022 15:45
@clangenb clangenb changed the title Polkadot v0.9.29 + ed25519-zebra patch Polkadot v0.9.29 Sep 29, 2022
@OverOrion OverOrion marked this pull request as ready for review October 20, 2022 11:52
@OverOrion OverOrion requested a review from haerdib October 20, 2022 14:40
Copy link
Contributor

@haerdib haerdib left a comment

Choose a reason for hiding this comment

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

LGTM!

One remark: unlike all the other repositories, we keep the api-client master pointing to master branches of substrate. That's because multiple, independent projects are using this client.

So keep this branch as it is (polkadot-v0.9.29 should point to the equivalent substrate branch), but create a new PR with the same changes but with master branches. With
cargo update -p sp-core --precise COMMITHASH you should be able to choose a commit with the same level as the polkadot v0.9.29 branch)

@OverOrion
Copy link
Contributor

Thank you @haerdib, opened the to-be-merged PR: #289.

@OverOrion OverOrion closed this Oct 21, 2022
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.

4 participants