-
Notifications
You must be signed in to change notification settings - Fork 10.3k
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
Update Deploying to Heroku docs #14811
Conversation
The instructions for adding buildpacks here are going to result in errors and confusion. `heroku buildpacks:set` sets a single buildpack so running it twice will override the first one. I've changed the instructions to still run `heroku buildpacks:set heroku/nodejs` first which will ensure `heroku/nodejs` is the only buildpack set, then run `heroku buildpacks:add https://github.com/heroku/heroku-buildpack-static.git` so it will add it as a second buildpack. (the difference being `buildpacks:add` in the second command) `app.json` is only honored for the heroku button and review apps so it should be specified as optional. I replaced the ambiguous "sometimes this doesn't work" comment with more details about when you might want to use it and a link to the heroku platform api article which covers `app.json`.
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.
Thanks a lot!
Holy buckets, @gpspake — we just merged your PR to Gatsby! 💪💜 Gatsby is built by awesome people like you. Let us say “thanks” in two ways:
If there’s anything we can do to help, please don’t hesitate to reach out to us: tweet at @gatsbyjs and we’ll come a-runnin’. Thanks again! |
The instructions for adding buildpacks here are going to result in errors and confusion. `heroku buildpacks:set` sets a single buildpack so running it twice will override the first one. I've changed the instructions to still run `heroku buildpacks:set heroku/nodejs` first which will ensure `heroku/nodejs` is the only buildpack set, then run `heroku buildpacks:add https://github.com/heroku/heroku-buildpack-static.git` so it will add it as a second buildpack. (the difference being `buildpacks:add` in the second command) `app.json` is only honored for the heroku button and review apps so it should be specified as optional. I replaced the ambiguous "sometimes this doesn't work" comment with more details about when you might want to use it and a link to the heroku platform api article which covers `app.json`.
The instructions for adding buildpacks here are going to result in errors and confusion. `heroku buildpacks:set` sets a single buildpack so running it twice will override the first one. I've changed the instructions to still run `heroku buildpacks:set heroku/nodejs` first which will ensure `heroku/nodejs` is the only buildpack set, then run `heroku buildpacks:add https://github.com/heroku/heroku-buildpack-static.git` so it will add it as a second buildpack. (the difference being `buildpacks:add` in the second command) `app.json` is only honored for the heroku button and review apps so it should be specified as optional. I replaced the ambiguous "sometimes this doesn't work" comment with more details about when you might want to use it and a link to the heroku platform api article which covers `app.json`.
Description
The instructions for adding buildpacks here could result in some errors and confusion.
heroku buildpacks:set
sets a single buildpack so running it twice will override the first one.I've changed the instructions to still run
heroku buildpacks:set heroku/nodejs
first which will ensureheroku/nodejs
is the only buildpack set, then runheroku buildpacks:add https://github.com/heroku/heroku-buildpack-static.git
so it will add it as a second buildpack. (the difference beingbuildpacks:add
in the second command)app.json
is only honored for the heroku button and review apps so it should be specified as optional.I replaced the ambiguous "sometimes this doesn't work" comment with more details about when you might want to use it and a link to the heroku platform api article which covers
app.json
.