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

Disable Geth tests in CI #2744

Merged
merged 2 commits into from
Oct 18, 2019
Merged

Disable Geth tests in CI #2744

merged 2 commits into from
Oct 18, 2019

Conversation

vbaranov
Copy link
Member

@vbaranov vbaranov commented Sep 30, 2019

Motivation

In order to decrease the number of tests without losing the efficiency of CI, it is suggested to remove the tests, that we start on Geth node. We have the same tests, starting on Parity node. If a new feature will be introduced specifically to Geth node, those tests suggested to be run locally.

Changelog

Tests, that starts on Geth node were commented in CI

Checklist for your PR

@coveralls
Copy link

coveralls commented Sep 30, 2019

@vbaranov vbaranov self-assigned this Sep 30, 2019
@vbaranov vbaranov added the ready for review This PR is ready for reviews. label Sep 30, 2019
@vbaranov vbaranov merged commit 2fe6112 into master Oct 18, 2019
@vbaranov vbaranov deleted the vb-disable-geth-tests branch October 18, 2019 13:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for review This PR is ready for reviews.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants