-
-
Notifications
You must be signed in to change notification settings - Fork 1.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
Remove deprecated heroku gem #3258
Conversation
bricesanchez
commented
Oct 18, 2016
We need this to be present, it's used in the generator, and I'm pretty sure it's had that deprecation message for a long time now. |
It's provide by the toolbelt now and our guide explicit that we need to have the toolbelt. |
My major complaint is that if people specify --heroku it won't automatically set things up, it'll fail. Maybe we should abort if the Heroku command isn't available? |
I've tried something, let me know if it's ok for you |
friendly ping @parndt |
fail <<-ERROR | ||
\033[31m[ABORTING]\033[0m Heroku Toolbelt is not installed. Please re-start the installer after installing at:\nhttps://devcenter.heroku.com/articles/heroku-command-line | ||
ERROR | ||
end |
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.
heroku -v
doesn't work for me, but heroku --version
does.
How about:
unless system("heroku --version")
fail <<-ERROR
\033[31m[ABORTING]\033[0m Heroku Toolbelt is not installed. Please re-start the installer after installing at:\nhttps://devcenter.heroku.com/articles/heroku-command-line
ERROR
end
And can we put the check inside a method like heroku_toolbelt_missing?
def heroku_toolbelt_missing?
!!system("heroku --version")
end
Then:
if heroku_toolbelt_missing?
fail <<-ERROR
\033[31m[ABORTING]\033[0m Heroku Toolbelt is not installed. Please re-start the installer after installing at:\nhttps://devcenter.heroku.com/articles/heroku-command-line
ERROR
end
755a8dc
to
2db8ff9
Compare
The `heroku` gem has been deprecated and replaced with the Heroku Toolbelt.
2db8ff9
to
dfe3e2b
Compare
🆙 😄 |
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!