Skip to content
This repository has been archived by the owner on Aug 5, 2022. It is now read-only.

Move to Heroku #11

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

Move to Heroku #11

wants to merge 3 commits into from

Conversation

bfreeds
Copy link

@bfreeds bfreeds commented Jan 8, 2020

This PR removes the configuration for Dokku and configures for Heroku deployment, which is the recommended route for deploying a Hubot. It follows Hubot's documentation for Heroku deployment.

This configures plannerbot for Heroku by:

  • Removing package-lock.json since having both yarn.lock and package-lock.json is messy (and blocks Heroku from deploying).
  • Removing circleci continuous deployment configuration for Dokku.
  • Updating documentation to reflect Heroku deploy.

Not reflected in the diff of this PR is the Heroku setup itself, which was done manually:

  • Created heroku app labs-plannerbot
  • Configured with environment variables
  • Installed Redis Cloud add-on (free for 30mb/month)
  • Connected to autodeploy from master:
    image .

Closes #6 and #8 .

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Move to Heroku
1 participant