Skip to content
This repository has been archived by the owner on Mar 14, 2019. It is now read-only.

convert to puma, add circle, Dockerfile #49

Merged
merged 9 commits into from
Oct 11, 2016
Merged

Conversation

jlambert121
Copy link
Contributor

Convert unicorn -> puma
Add circle.yml
Add Dockerfile

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)
Copy link
Contributor

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.

Copy link
Contributor

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"

Copy link
Contributor

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

Copy link
Contributor

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

@joecannatti joecannatti merged commit 8e02644 into master Oct 11, 2016
@joecannatti joecannatti deleted the circle_docker branch October 11, 2016 17:34
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.

3 participants