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

Make integration tests part of CI workflow #723

Merged
merged 22 commits into from
Nov 18, 2020
Merged

Conversation

kmazurek
Copy link
Contributor

@kmazurek kmazurek commented Oct 22, 2020

Resolves: golemfactory/goth#225

The recently added integration-test workflow is currently not part of blocking checks on PRs. This makes integration tests part of the rust CI workflow. The current running time for these tests on Actions is ~4:30.

@kmazurek kmazurek requested a review from prekucki as a code owner October 22, 2020 11:53
@kmazurek kmazurek requested a review from a team October 22, 2020 11:53
@kmazurek kmazurek self-assigned this Oct 22, 2020
@kmazurek kmazurek force-pushed the km/ci-integration-tests branch from 704f80b to f7e73ad Compare October 23, 2020 12:37
@kmazurek kmazurek force-pushed the km/ci-integration-tests branch from f7e73ad to 4a7ff99 Compare October 23, 2020 12:56
prekucki
prekucki previously approved these changes Oct 28, 2020
@kmazurek kmazurek requested a review from tworec November 12, 2020 14:39
@prekucki prekucki self-requested a review November 12, 2020 15:55
prekucki
prekucki previously approved these changes Nov 12, 2020
tworec
tworec previously approved these changes Nov 13, 2020
prekucki
prekucki previously approved these changes Nov 14, 2020
@kmazurek kmazurek dismissed stale reviews from prekucki and tworec via 9295052 November 16, 2020 12:33
@kmazurek kmazurek merged commit bdf8e5c into master Nov 18, 2020
@kmazurek kmazurek deleted the km/ci-integration-tests branch November 18, 2020 15:14
mfranciszkiewicz pushed a commit that referenced this pull request Nov 20, 2020
Co-authored-by: Przemysław Rekucki <[email protected]>
Co-authored-by: Piotr Chromiec <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement CI for yagna repo using TestHarness
4 participants