-
Notifications
You must be signed in to change notification settings - Fork 535
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
migrate from Travis CI to GitHub Actions CI #516
Conversation
@reactive-streams/contributors Any thoughts or objections towards switching to GH Actions CI instead of TravisCI? |
@viktorklang can you merge this? |
env: | ||
global: | ||
- TERM=dumb | ||
- DEFAULT_TIMEOUT_MILLIS=300 |
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.
@sullis What about these TCK ENV vars?
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.
Travis CI recently put many of us to their free but severely limited OSS plan (1000 build minutes credit and the need to keep renegotiating every time it runs out). I'd say for the CI build needs of this project, GitHub Actions is quite adequate. |
Signed-off-by: Sean C. Sullivan <github@seansullivan.com>
Co-authored-by: Oleh Dokuka <shadowgun@i.ua> Signed-off-by: Sean C. Sullivan <github@seansullivan.com>
ping @akarnokd |
can you merge this? @akarnokd |
I don't have the rights. |
@rkuhn Would you be able to merge this, I'm currently not on a machine
where I have access.
|
done |
build failed: https://github.com/reactive-streams/reactive-streams-jvm/runs/2827968925?check_suite_focus=true#step:5:31 Any gradle folks around to fix it? Otherwise I’ll need to revert. |
No description provided.