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

Docker #43

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

Docker #43

wants to merge 11 commits into from

Conversation

jcsumlin
Copy link
Owner

No description provided.

J_C___ and others added 11 commits March 13, 2021 23:15
Created Dockerfile that builds the app
Updated app to use environment variables
removed spammy commands in animal.py
Updated birthday.py to be able to grant a birthday role!
updated birthdays to use proper user lookup functions
added subtract method for bookclub.py
Made it so if statcordpost.py doesnt have env var set it will not try and post.

Update docker image to not need build args. These can be injected on container run
Fixed Ping command
Updated README.md to remind me of what needs to be done.
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.

1 participant