Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

derive Serialize/Deserialize for BeefyAuthoritySet #12516

Merged
merged 1 commit into from
Oct 18, 2022

Conversation

serban300
Copy link
Contributor

@serban300 serban300 commented Oct 18, 2022

Related to: paritytech/parity-bridges-common#2469

We are working on implementing BEEFY finality in the bridges project and as part of this we'll need to have the initial BeefyAuthoritySet in the GenesisConfig structure. In order to do this we need BeefyAuthoritySet to implement serde serialize/deserialize.

@serban300 serban300 added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit labels Oct 18, 2022
@serban300 serban300 self-assigned this Oct 18, 2022
@bkchr
Copy link
Member

bkchr commented Oct 18, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Statuses failed for 263cc3f

@bkchr
Copy link
Member

bkchr commented Oct 18, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Statuses failed for 263cc3f

@acatangiu
Copy link
Contributor

bot rebase

@bkchr bkchr merged commit 281bcd4 into paritytech:master Oct 18, 2022
@paritytech-processbot
Copy link

Error: Command 'Command { std: "git" "push" "--porcelain" "serban300" "small_fixes", kill_on_drop: false }' failed with status Some(1); output: error: failed to push some refs to 'https://x-access-token:${SECRET}@github.com/serban300/substrate.git'

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes C1-low PR touches the given topic and has a low impact on builders. D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants