-
Notifications
You must be signed in to change notification settings - Fork 18
convert to puma, add circle, Dockerfile #49
Conversation
b15dc86
to
c348891
Compare
on_worker_boot do | ||
# Worker specific setup for Rails 4.1+ | ||
# See: https://devcenter.heroku.com/articles/deploying-rails-applications-with-the-puma-web-server#on-worker-boot | ||
if defined?(ActiveRecord::Base) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This app doesn't use a database, so this might not be needed.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hmm, though I can see it was in the unicorn config, so let's leave it for "changing less things"
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LOL, right. going to remove it
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry, didn't see your second comment. Will leave it
Convert unicorn -> puma
Add
circle.yml
Add Dockerfile