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

Dockerized #39

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

Dockerized #39

wants to merge 7 commits into from

Conversation

TechForze
Copy link

Hi,

I moved the data, added Dockerfile, so it can work in my docker setup. Thanks!

@eschava
Copy link
Owner

eschava commented Mar 25, 2019

Sorry, but why it was required to move all these files to data folder?

@TechForze
Copy link
Author

So it is possible to single mount that folder in docker, with the config and actions with newly recorded actions.
Maybe there are other solutions to save the recorded actions or config with docker, but this is one is suited best for me.
Maybe you have another solution for this?

Copy link

@tomcollis tomcollis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These changes all look fine, and I would certainly prefer to run this as a docker container than just as a script on a Linux machine.

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.

3 participants