New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Adds bin/setup and Procfile #88
Conversation
d6b407b
to
c99a928
Compare
c99a928
to
91c6705
Compare
1 similar comment
@@ -0,0 +1,2 @@ | |||
web: bundle exec puma -t 5:5 -p ${PORT:-3000} -e ${RACK_ENV:-development} |
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.
We also have this configuration in config/puma.rb
, using it in bin/cf-start
.
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.
Procfile is required for foreman and Heroku; does this collide with config/puma.rb ?
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.
What I'm suggesting is something like:
web: bundle exec puma -C config/puma.rb
...as in bin/cf-start
.
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.
AH -- yes, I think that is the correct approach. I will update!
Updated Procfile @jmcarp and tested on Heroku 🚀 |
* Adds bin/setup and procfile * Updates README formatting * Updates Procfile to read from puma.rb
* Adds bin/setup and procfile * Updates README formatting * Updates Procfile to read from puma.rb
* Adds bin/setup and procfile * Updates README formatting * Updates Procfile to read from puma.rb
* Adds bin/setup and procfile * Updates README formatting * Updates Procfile to read from puma.rb
Includes bin/setup and Procfile from my original PR