UniversityOfNottingham / devbot

0 stars 0 forks source link

Remove unnecessary environment variables from dockerfile #1

Closed beforan closed 7 years ago

beforan commented 7 years ago

I'm pretty sure slack integration doesn't require half the environment variables anymore, so we should determine which ones we can remove, and chuck them from the dockerfile.

beforan commented 7 years ago

The shell script I'm using atm to run the container sets the following environment variables:

But I think even some of those are unnecessary, as the integration nowadays specifies the name, and is added to a team. If I had to guess I'd say only the Slack token is needed.

beforan commented 7 years ago

confirmed only HUBOT_SLACK_TOKEN is required.