Add Docker support #9
Conversation
|
Hi! Thank you for your PR! I don't really see why someone would be interested in running the script inside Docker, but I think it's positive to have the option available for those who find it useful! To be honest, I would give instructions to run the script simply with docker, not docker compose. The "Method 3" in the README is bigger than the others ("###" vs "####"). I can do these changes myself and commit to your PR directly if you are OK with it. |
|
That's fine with me! Personally I run it in Docker myself since I find it easier to provision and maintain :) |
|
Hi, Sorry for the delay! I will proceed with the changes then. I like the idea of adding notification support. But personally, I don't know of Apprise. Maybe we could add support for generic webhooks? Like adding the payload and url in the config toml. I think that would be better. |
|
Ah shit I meant to open another PR for that... didn't realize I pushed to this branch, sorry 😅 That could work! Then in that case you can also discard my commit when you make the other changes |
This PR adds support to build and run the image in Docker