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

Dockerfile setup with Travis CI check and documentation update #210

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

mbocevski
Copy link
Contributor

Fixes #110
Fixes #136
Fixes #163

Considering datapusher is pretty-much a standalone component, it makes sense to include a Dockerfile, which can easily be extended to support all kinds of docker based deployment scenarios. The dockerfile is based on Alpine and it's using a multistage approach. Also to ensure that it doesn't break a CI test is included in Travis and basic docs.

@gustavovalverde
Copy link

gustavovalverde commented Sep 25, 2021

I see this commit could help us a lot to have an up to date version of datapusher. I could help automate the build and submit process to DockerHub using Github Actions, please let me know to submit a PR ASAP.

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.

2 participants