-
-
Notifications
You must be signed in to change notification settings - Fork 515
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
feature: move to github action #242 #257
Conversation
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.
Hi @Bablzz, thank you for this great contribution. I added a few nits about names, but on the other hand it will LGTM after hearing your opinion.
Thanks! 👋
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.
@gianarb I think we can keep travis for a while, comparing executions, and once we are happy with GH, then move on removing Travis descriptor. |
THanks @Bablzz !!! and thanks @mdelapenya for your review! |
@gianarb @mdelapenya My pleasure! 🤝 |
Hello everyone!
In this MR I add github action. There are the same steps as in .travis.yml