Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Create Dockerfile for MusicBot hosting #1433

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

Conversation

enigodupont
Copy link

@enigodupont enigodupont commented Sep 14, 2023

This pull request...

  • Fixes a bug
  • Introduces a new feature
  • Improves an existing feature
  • Boosts code quality or performance

Description

Create a Dockerfile for MusicBot

Purpose

We should support running this on servers via docker.
We should support running this without installing Java to the host machine.

Relevant Issue(s)

N/A


download() {
if [ $DOWNLOAD == true ]; then
if [ "$DOWNLOAD" = true ]; then
Copy link
Author

Choose a reason for hiding this comment

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

shellcheck changes


# This will cause the script to run in a loop so that the bot auto-restarts
# when you use the shutdown command
LOOP=true
LOOP=${LOOP:-false}
Copy link
Author

Choose a reason for hiding this comment

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

Adding sane defaults for the script with overrides

@enigodupont
Copy link
Author

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants