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

Add GitHub Actions CI #156

Merged
merged 1 commit into from
Feb 6, 2021
Merged

Add GitHub Actions CI #156

merged 1 commit into from
Feb 6, 2021

Conversation

eumiro
Copy link
Contributor

@eumiro eumiro commented Feb 4, 2021

Since the main repo does not contain GHA, the badge won't be visible until the merge. You can see the CI run results at: https://github.com/eumiro/markovify/actions/runs/537978391

When everything works fine, maybe you can consider removing Travis.

@coveralls
Copy link

coveralls commented Feb 4, 2021

Coverage Status

Coverage remained the same at 100.0% when pulling 017eb84 on eumiro:add-gha into d2c9f71 on jsvine:master.

@jsvine jsvine merged commit 75a2590 into jsvine:master Feb 6, 2021
@jsvine
Copy link
Owner

jsvine commented Feb 6, 2021

Thanks, I like this PR. The vast majority of markovify was built before GitHub Actions were available. But now that it's an option, it seems like an improvement over Travis.

@eumiro eumiro deleted the add-gha branch February 6, 2021 16:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants