-
Notifications
You must be signed in to change notification settings - Fork 32
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 #3
Conversation
11d9920
to
20d1d1f
Compare
20d1d1f
to
c322164
Compare
@cdesiniotis PTAL |
c322164
to
3f05c38
Compare
c4f7a76
to
7b35383
Compare
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.
Some questions.
7b35383
to
cf632ca
Compare
6f99e5c
to
97ad273
Compare
Signed-off-by: Carlos Eduardo Arango Gutierrez <[email protected]>
97ad273
to
2988409
Compare
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.
Since the builds are working for 550 and precompiled, I am okay to merge this. We can address any follow-ups / improvements in a follow-up.
Also remember that gitlab pipelines will continue to work in the mean time, so we are double covered for now |
This patch adds 2 github actions one for pull-request and one for push (post-merge)