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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Replace TravisCI with GitHub Actions #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

ewels
Copy link

@ewels ewels commented Jul 21, 2020

Hi! Great talk at BOSC2020 馃帀

Following our brief chat on BOSC Discord, I thought I would offer some help in switching from Travis CI to GitHub Actions. We've made this change in the @nf_core community because GitHub Actions is automatically enabled for new GitHub repositories and forks and doesn't require users to do any additional steps. In our experience it's also faster and gives a nicer interface in GitHub pull-requests.

Whilst I was here, I also added a workflow to build the Docker image and publish it to the GitHub Package Registry. The same logic applies here - this should work out of the box for any users, and keeps all resources attached to a single repository.

Please note that I have not run any of the original Travis CI tests or used bionitio previously, so this code is purely written by reading what you already had set up. It will probably need some work to (a) get it working and (b) get it to fit in to the bionitio community nicely for other languages.

I hope this helps!

Phil

Improvements for the Docker GitHub Action:

* Use an automated actions variable for the repo name, instead of hardcoding
* Run the functional tests after building and pushing
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

1 participant