-
Notifications
You must be signed in to change notification settings - Fork 632
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 GitHub actions #1241
Add GitHub actions #1241
Conversation
Codecov Report
@@ Coverage Diff @@
## main #1241 +/- ##
==========================================
- Coverage 91.83% 91.34% -0.50%
==========================================
Files 136 142 +6
Lines 9056 9926 +870
==========================================
+ Hits 8317 9067 +750
- Misses 739 859 +120
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looks good! circleci kinda sucks
Co-authored-by: dyllan <[email protected]>
9969bf0
to
459eabe
Compare
Co-authored-by: dyllan <[email protected]>
…iveloopai/Hub into feature/github-actions-build
🚀 🚀 Pull Request
Checklist:
coverage-rate
upChanges
This PR brings GitHub actions support in Hub repository. All the checks (including complex) tests were taken from appropriate CircleCI workflow.
Note
Since GitHub actions workers (including paid ones) have only 2 CPU cores, the worker number was reduced from 4 to 2.
Kaggle tests were disabled by removing
--kaggle
flag from tests.After merge
After the PR is merged, the build must be included in settings area as a required step for PR checks