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

Update RequestResponseConfig interface to match substrate #5849

Merged
merged 6 commits into from
Aug 5, 2022

Conversation

dmitry-markin
Copy link
Contributor

@dmitry-markin dmitry-markin commented Aug 1, 2022

Companion to paritytech/substrate#11938. This PR leaves the protocol names in Polkadot as they are and is here just to merge the Substrate changes. Separate PR is needed to change Polkadot protocol names to also include the genesis hash and fork id.

@dmitry-markin dmitry-markin added A0-please_review Pull request needs code review. B0-silent Changes should not be mentioned in any release notes D3-trivial 🧸 PR contains trivial changes in a runtime directory that do not require an audit. labels Aug 1, 2022
@dmitry-markin dmitry-markin added the C1-low PR touches the given topic and has a low impact on builders. label Aug 1, 2022
@dmitry-markin dmitry-markin added the A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. label Aug 2, 2022
@paritytech-processbot
Copy link

Waiting for commit status.

@dmitry-markin dmitry-markin added A8-mergeoncegreen and removed A0-please_review Pull request needs code review. labels Aug 5, 2022
@paritytech-processbot
Copy link

Merge cancelled due to error. Error: Statuses failed for 7f88793

@paritytech-processbot
Copy link

Waiting for commit status.

@paritytech-processbot paritytech-processbot bot merged commit 1bd41fc into master Aug 5, 2022
@paritytech-processbot paritytech-processbot bot deleted the dm-substrate-protocol-names branch August 5, 2022 10:37
ordian added a commit that referenced this pull request Aug 5, 2022
* master:
  Update RequestResponseConfig interface to match substrate (#5849)
ordian added a commit that referenced this pull request Aug 5, 2022
* master:
  Update RequestResponseConfig interface to match substrate (#5849)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A4-companion A PR that needs a companion PR to merge in parallel for one of its downstream dependencies. 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.

3 participants