Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Skip beacon chain core spec tests until v0.12 changes are complete #5939

Merged
merged 1 commit into from
May 21, 2020

Conversation

0xKiwi
Copy link
Contributor

@0xKiwi 0xKiwi commented May 21, 2020

What type of PR is this?
Test skipping

What does this PR do? Why is it needed?
This PR skips all of the spec tests in beacon-chain/core/ to not pollute buildkite with test failures while we work on the branch. Once #5935 is complete, this will be reverted.

This PR does not add skips to shared/bls/spectest and shared/params/spectest, please let me know if either of those are needed.

@0xKiwi 0xKiwi requested a review from a team as a code owner May 21, 2020 06:07
@0xKiwi 0xKiwi requested review from rauljordan, shayzluf and prestonvanloon and removed request for a team May 21, 2020 06:07
@0xKiwi 0xKiwi added the Ready For Review A pull request ready for code review label May 21, 2020
@terencechain terencechain merged commit 7578775 into v0.12 May 21, 2020
@delete-merged-branch delete-merged-branch bot deleted the skip-spec-tests branch May 21, 2020 06:11
@terencechain terencechain changed the title Skip spec tests until v0.12 changes are complete Skip beacon chain core spec tests until v0.12 changes are complete May 21, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Ready For Review A pull request ready for code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants