Skip to content
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

Merged
merged 3 commits into from
Nov 10, 2016
Merged

Remove deprecated heroku gem #3258

merged 3 commits into from
Nov 10, 2016

Conversation

bricesanchez
Copy link
Member

Fetching: heroku-3.43.12.gem (100%)
 !    The `heroku` gem has been deprecated and replaced with the Heroku Toolbelt.
 !    Download and install from: https://toolbelt.heroku.com
 !    For API access, see: https://github.com/heroku/heroku.rb

@parndt
Copy link
Member

parndt 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.

@bricesanchez
Copy link
Member Author

It's provide by the toolbelt now and our guide explicit that we need to have the toolbelt.

@parndt
Copy link
Member

parndt commented Oct 18, 2016

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?

@bricesanchez
Copy link
Member Author

I've tried something, let me know if it's ok for you

@bricesanchez
Copy link
Member Author

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
Copy link
Member

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

@bricesanchez bricesanchez force-pushed the bugfix/deprecated-heroku-gem branch 2 times, most recently from 755a8dc to 2db8ff9 Compare November 4, 2016 02:09
Brice Sanchez and others added 3 commits November 8, 2016 21:41
@bricesanchez bricesanchez force-pushed the bugfix/deprecated-heroku-gem branch from 2db8ff9 to dfe3e2b Compare November 9, 2016 02:41
@bricesanchez
Copy link
Member Author

🆙 😄

Copy link
Member

@parndt parndt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@parndt parndt merged commit ae14fa0 into master Nov 10, 2016
@parndt parndt deleted the bugfix/deprecated-heroku-gem branch November 10, 2016 04:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants