-
Notifications
You must be signed in to change notification settings - Fork 14
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
Add continuous integration unit testing through GH Actions #160
Conversation
Also, this PR would replace PR #22. I think GH Actions is a better solution than Travis at this point. |
Ok, solved error above which was due to a deprecated version of a package that was installed due to package conflicts. Removing the All tests (including Compute Studio app tests) are passing locally for me. Looking into why failures on the GH Action. |
Took a few tries to figure out how to properly install |
Codecov Report
@@ Coverage Diff @@
## master #160 +/- ##
=========================================
Coverage ? 81.83%
=========================================
Files ? 21
Lines ? 1448
Branches ? 0
=========================================
Hits ? 1185
Misses ? 263
Partials ? 0
Flags with carried forward coverage won't be shown. Click here to find out more. Continue to review full report at Codecov.
|
@hdoupe Thanks for suggestion - one line worked! |
Awesome, thanks for stepping up and figuring out how to get this to work. |
This PR addresses Issue #106 |
Thanks for working on this, @jdebacker! LGTM. Merging. |
This PR added a GH Action to run unit tests on pushes.
I have a work in progress label because I can't get unit tests to run locally. I just rebuilt the
toxbrain-dev
environment and when I runpytest
, I get:Once I am able to confirm tests run locally, I'll remove the WIP label.