-
Notifications
You must be signed in to change notification settings - Fork 97
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
Docker support #89
Comments
Thank @undernewmanagement , let me check it and create a PR. |
Isn't CMD/ENTRYPOINT missing there? |
I couldn't find any Dockerfile in this repo, and the fork linked in this ticket is giving me a 404. For my own use, I made one up...
and build it
and pushed it
and now I store a config in my host machine's ssh folder which I think is a fairly safe place to keep secrets, and I use this command to fire up sclack
It works great, so just thought I would share on this ticket in case it is useful. Only caveat I have found so far, is that the browser feature would not work, as the running container does not have access to the host machine browser. |
I've got a branch on a fork where I've wrapped things up in a Dockerfile.
https://github.com/undernewmanagement/sclack/commit/101ce775e229a1d959010dcdba482dc350d9a6f5
Currently it only installs the non-development version of requirements.txt
With blessing of admin and community I'm happy to submit pull request for review.
The text was updated successfully, but these errors were encountered: