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

pvf-precheck: Candidate Validation Changes#4409

Merged
pepyakin merged 3 commits intomasterfrom
pep-precheck-candidate-validation
Nov 30, 2021
Merged

pvf-precheck: Candidate Validation Changes#4409
pepyakin merged 3 commits intomasterfrom
pep-precheck-candidate-validation

Conversation

@pepyakin
Copy link
Copy Markdown
Contributor

@pepyakin pepyakin commented Nov 29, 2021

This PR is a part of #3211.

As part of this PR we are integrating the new validation host function,
that is precheck_pvf, into the candidate-validation subsystem, which will
be the main entrypoint for pre-checking subsystem (To be introduced by
PRs down the stack).

skip check-dependent-cumulus

@pepyakin
Copy link
Copy Markdown
Contributor Author

pepyakin commented Nov 29, 2021

Current dependencies on/for this PR:

This comment was auto-generated by Graphite and will continue to be automatically updated while this PR remains open.

@github-actions github-actions bot added the A0-please_review Pull request needs code review. label Nov 29, 2021
@pepyakin pepyakin added 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. labels Nov 29, 2021
Base automatically changed from pep-primitives-pvf-check-statement to master November 30, 2021 10:32
@pepyakin pepyakin force-pushed the pep-precheck-candidate-validation branch 3 times, most recently from 7afadb3 to 19f746f Compare November 30, 2021 12:07
@pepyakin pepyakin requested review from drahnr and ordian November 30, 2021 15:36
@pepyakin pepyakin force-pushed the pep-precheck-candidate-validation branch from 19f746f to 9c18425 Compare November 30, 2021 15:48
@pepyakin pepyakin merged commit 4a2d498 into master Nov 30, 2021
@pepyakin pepyakin deleted the pep-precheck-candidate-validation branch November 30, 2021 23:53
drahnr pushed a commit that referenced this pull request Dec 1, 2021
* pvf-precheck: Candidate Validation Changes

Co-authored-by: Chris Sosnin <chris125_@live.com>

* Rename `MockValidationBackend` and specialize it

* Add pre-check tests

Co-authored-by: Chris Sosnin <chris125_@live.com>
drahnr pushed a commit that referenced this pull request Dec 1, 2021
* pvf-precheck: Candidate Validation Changes

Co-authored-by: Chris Sosnin <chris125_@live.com>

* Rename `MockValidationBackend` and specialize it

* Add pre-check tests

Co-authored-by: Chris Sosnin <chris125_@live.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. 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