Skip to content
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

tests: use GitHub Actions for CI #1295

Merged
merged 1 commit into from
Nov 6, 2020
Merged

Conversation

ploxiln
Copy link
Member

@ploxiln ploxiln commented Nov 6, 2020

also add go-1.14 and go-1.15 to test matrix

TODO: try to add back coveralls

@ploxiln
Copy link
Member Author

ploxiln commented Nov 6, 2020

sometimes this starts working right away, sometimes this file has to exist in master before it'll trigger on and use the changes in the pull requests ...

also add go-1.14 and go-1.15 to test matrix

keep Travis-CI temporarily
@ploxiln ploxiln changed the title tests: switch from Travis-CI to GitHub Actions tests: use GitHub Actions for CI Nov 6, 2020
@ploxiln
Copy link
Member Author

ploxiln commented Nov 6, 2020

@mreiferson @jehiah do you mind if I merge this in order to get GitHub Actions enabled, and then finish it up in a follow-up PR? (changes to the actions take effect in the PR, but it has to exist in master to be enabled, I think?)

see also https://docs.travis-ci.com/user/migrate/open-source-repository-migration/#q-what-will-happen-to-travis-ciorg-after-december-31st-2020

@ploxiln
Copy link
Member Author

ploxiln commented Nov 6, 2020

thanks

@ploxiln ploxiln merged commit 500eae6 into nsqio:master Nov 6, 2020
@ploxiln
Copy link
Member Author

ploxiln commented Nov 6, 2020

@mreiferson
Copy link
Member

thanks for this ❤️

@ploxiln ploxiln deleted the github_actions branch December 4, 2020 09:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants