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

gate approval-checking logic #2470

Merged
merged 1 commit into from
Feb 18, 2021
Merged

Conversation

rphmeier
Copy link
Contributor

@rphmeier rphmeier commented Feb 17, 2021

This feature-gates all code for approval checking behind an approval-checking feature.

@rphmeier rphmeier 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 Feb 17, 2021
@rphmeier
Copy link
Contributor Author

bot merge

@ghost
Copy link

ghost commented Feb 17, 2021

Waiting for commit status.

@ghost
Copy link

ghost commented Feb 17, 2021

Merge failed: "At least 1 approving review is required by reviewers with write access."

@ordian ordian merged commit 0e0d051 into master Feb 18, 2021
@ordian ordian deleted the rh-conditional-enable-approval-checking branch February 18, 2021 07:05
rphmeier added a commit that referenced this pull request Feb 18, 2021
bkchr pushed a commit that referenced this pull request Feb 19, 2021
ghost pushed a commit that referenced this pull request May 3, 2021
* Update branch

* Make it compile

* Compile

* gate approval-checking logic (#2470)

* Fix build

* Updates

* Fix merge

* Adds missing crate

* Companion for Substrate#8386

paritytech/substrate#8386

* Fix fix fix

* Fix

* Fix compilation

* Rewrite to `ParachainsInherentDataProvider`

* Make it compile

* Renamings

* Revert stuff

* Remove stale file

* Guide updates

* Update node/core/parachains-inherent/src/lib.rs

Co-authored-by: Andronik Ordian <[email protected]>

* Update node/core/parachains-inherent/src/lib.rs

Co-authored-by: Andronik Ordian <[email protected]>

* Apply suggestions from code review

* Reset accidental changes

* More

* Remove stale file

* update Substrate

Co-authored-by: Robert Habermeier <[email protected]>
Co-authored-by: Andronik Ordian <[email protected]>
Co-authored-by: parity-processbot <>
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.

2 participants