Skip to content

Babe epoch newtype#1596

Merged
davxy merged 2 commits into
paritytech:masterfrom
davxy:babe-epoch-newtype
Sep 17, 2023
Merged

Babe epoch newtype#1596
davxy merged 2 commits into
paritytech:masterfrom
davxy:babe-epoch-newtype

Conversation

@davxy

@davxy davxy commented Sep 16, 2023

Copy link
Copy Markdown
Member

Removal of verbatim duplication of BABE's Epoch struct in the client.

I think is better to have one single definition and wrap the primitive Epoch in a newtype (required because we need to implement the Epoch trait).

@davxy davxy requested a review from andresilva as a code owner September 16, 2023 10:41
@davxy davxy self-assigned this Sep 16, 2023
@davxy davxy added the T0-node This PR/Issue is related to the topic “node”. label Sep 16, 2023
@davxy davxy requested a review from a team September 16, 2023 10:42
@davxy davxy merged commit c7dbfc2 into paritytech:master Sep 17, 2023
@davxy davxy deleted the babe-epoch-newtype branch December 1, 2023 15:47
bgallois pushed a commit to duniter/duniter-polkadot-sdk that referenced this pull request Mar 25, 2024
Removal of verbatim duplication of BABE's `Epoch` struct in the client.

I think is better to have one single definition and wrap the primitive
`Epoch` in a newtype (required because we need to implement the `Epoch`
trait).
bkchr pushed a commit that referenced this pull request Apr 10, 2024
* remove without_storage_info from pallet-bridge-parachains

* fix benchmarks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

T0-node This PR/Issue is related to the topic “node”.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants