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

properly gate sanity check #3684

Merged
merged 1 commit into from
Aug 21, 2021
Merged

properly gate sanity check #3684

merged 1 commit into from
Aug 21, 2021

Conversation

rphmeier
Copy link
Contributor

@rphmeier rphmeier commented Aug 21, 2021

This is only relevant when passing data through to disputes, and is noisy due to #3678

@rphmeier rphmeier 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. labels Aug 21, 2021
@rphmeier rphmeier merged commit a1b08f9 into master Aug 21, 2021
@rphmeier rphmeier deleted the rh-gate-sanity-check branch August 21, 2021 02:11
GopherJ pushed a commit to parallel-finance/polkadot that referenced this pull request Aug 23, 2021
bkchr pushed a commit that referenced this pull request Aug 23, 2021
* Bypass chain-selection subsystem until disputes are enabled. (#3676)

* cleanup of rob's patch

* depend on feature disputes, both lag and query

* typo

* make it work

TODO move the fallback into selection

* option

* Update node/service/src/relay_chain_selection.rs

* remove trait implementation and fix errors

* hotfix: tests

* fmt

* remove unused trait

* make pub(crate) for test

* disable tests with disputed blocks

* Fix warnings

Co-authored-by: Robert Habermeier <rphmeier@gmail.com>
Co-authored-by: Lldenaurois <Ljdenaurois@gmail.com>

* properly gate sanity check (#3684)

Co-authored-by: Bernhard Schuster <bernhard@ahoi.io>
Co-authored-by: Robert Habermeier <rphmeier@gmail.com>
Co-authored-by: Lldenaurois <Ljdenaurois@gmail.com>
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.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants