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 continuous integration #77

Merged
merged 3 commits into from May 12, 2020
Merged

Add continuous integration #77

merged 3 commits into from May 12, 2020

Conversation

richelbilderbeek
Copy link
Contributor

@richelbilderbeek richelbilderbeek commented May 11, 2020

Dear Dr. Hyatt,

Prodigal appears to be awesome! Thanks so much for making it open source!

With this Pull Request, I allow this repo to be tested by the Travis CI continuous integraton (hence the name) service. In that way, one can be sure Prodigal works, especially when adding a cron job.

I am unsure how familiar you are with CI and if you are aware of the benefits it gives. I will happily share!

For sure, I always enjoy a build badge; an honest signal of quality. Here is it on my Fork (on the GoogleImports branch):

Build Status

On the master branch of the hyattpd repo, one can put this text in README.md:

Build Status

The Travis script installs and runs Prodigal and shows the output.

I hope you enjoy this Pull Request. Any questions I would enjoy to answer. I also volunteer to maintain the Travis script.

If you choose to reject this PR, sorry for the spam :-)

@hyattpd hyattpd merged commit 6ccc8d1 into hyattpd:GoogleImport May 12, 2020
@hyattpd
Copy link
Owner

hyattpd commented May 12, 2020

I'm not familiar with it at all, other than seeing the travis.yml files in everyone's repositories. I merged the request though.

@richelbilderbeek
Copy link
Contributor Author

I'm not familiar with it at all, other than seeing the travis.yml files in everyone's repositories. I merged the request though.

Thanks for the confidence! And I'll be happy to assist. Next step: activate the Travis CI service here (you can just sign in with GitHub).

And sure, I'll help!

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

2 participants