-
-
Notifications
You must be signed in to change notification settings - Fork 5
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
GHA: Run unit tests in CI #37
Conversation
763b205
to
55c2013
Compare
ea10f3f should be dropped before merge |
55c2013
to
42bf848
Compare
@bitsandfoxes @bruno-garcia |
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.
Awesome stuff!
- name: Run tests | ||
shell: bash | ||
timeout-minutes: 5 | ||
run: | |
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.
I assume this step fails if the tests fail?
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.
I'm not sure about (Works as expected).timeout-minutes
- docs don't specify if it fails the step. Need to check it.
According to docs, the process should return:
0 = all tests success
100 = ends with test failures
101 = ends with test warnings
bb09127
to
3abd66f
Compare
Run unit tests in CI using Windows and Linux runners. This will help us detect compilation and runtime issues early on.