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
Replaces Travis with GitHub Actions #791
Conversation
🎊 PR Preview 220a33e has been successfully built and deployed to https://quarkusio-quarkusio-github-io-preview-pr-791.surge.sh 🕐 Build time: 0.013s 🤖 By surge-preview |
f2d6073
to
20c6ba0
Compare
@@ -0,0 +1,26 @@ | |||
name: Build and deploy to GitHub Pages |
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.
as long as we can't guarantee there are a action resources free should we not just use gtihub actions for PR and still keep travis.yml for master/develop builds ?
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.
IMHO I don't think it's worth the trouble of having 2 CIs (and consequently 2 build scripts) for a single task.
It's highly likely that the time that Travis takes to build the master branch (due to the resource limits imposed) would take longer than the time it takes the GH Action to execute (even in high load scenarios).
So I'm -1 on that.
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.
Considering the issue we have with Travis, I think it's worth a try.
But we might have to revert this if it ends up being impractical.
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.
In the worst case scenario you can always run the build locally 😉
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 added instructions on how to deploy to GitHub pages locally if you ever need to
cf7e206
to
c1a4cdc
Compare
This removes the Travis descriptor and builds using GitHub actions and builds daily Add instructions on how to deploy to Github pages locally
OK, let's try this and see how it goes! Thanks! |
This removes the Travis descriptor and builds using GitHub actions and builds daily