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

Add attributes and documentation to fork_id #10746

Merged
merged 2 commits into from
Jan 28, 2022

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented Jan 28, 2022

cc #10463

The overwhelming majority of the time the fork_id will be empty, so let's not include the field in chain specs if that's the case.

@tomaka tomaka added A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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. labels Jan 28, 2022
Co-authored-by: Sacha Lansky <sacha@parity.io>
@bkchr
Copy link
Member

bkchr commented Jan 28, 2022

bot merge

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot
Copy link

Bot will approve on the behalf of @bkchr, since they are a team lead, in an attempt to reach the minimum approval count

@paritytech-processbot paritytech-processbot bot merged commit 1307647 into paritytech:master Jan 28, 2022
@tomaka tomaka deleted the fork-id-optional branch January 28, 2022 13:30
Wizdave97 pushed a commit to ComposableFi/substrate that referenced this pull request Feb 4, 2022
* Add attributes and documentation to fork_id

* Update client/chain-spec/src/chain_spec.rs

Co-authored-by: Sacha Lansky <sacha@parity.io>

Co-authored-by: Sacha Lansky <sacha@parity.io>
grishasobol pushed a commit to gear-tech/substrate that referenced this pull request Mar 28, 2022
* Add attributes and documentation to fork_id

* Update client/chain-spec/src/chain_spec.rs

Co-authored-by: Sacha Lansky <sacha@parity.io>

Co-authored-by: Sacha Lansky <sacha@parity.io>
ark0f pushed a commit to gear-tech/substrate that referenced this pull request Feb 27, 2023
* Add attributes and documentation to fork_id

* Update client/chain-spec/src/chain_spec.rs

Co-authored-by: Sacha Lansky <sacha@parity.io>

Co-authored-by: Sacha Lansky <sacha@parity.io>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A2-insubstantial Pull request requires no code review (e.g., a sub-repository hash update). 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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants