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

Resolves gemspec deprecation warnings on the Rails engine and Faye Extensions #213

Merged
merged 4 commits into from
Feb 25, 2016

Conversation

discorick
Copy link
Member

This was blocking our deployments to heroku as of this build pack deploy: heroku/heroku-buildpack-ruby#461

The deprecations look something like this:
image

On deploy they would literally prevent analytics-ruby from the rails engine gemspec from installing.

Take a look at the diff to see the offending code, pretty weird that 'TODO' strings would cause such a fuss.

s.homepage = "TODO"
s.summary = "TODO: Summary of Saas."
s.description = "TODO: Description of Saas."
s.homepage = "https://github.com/huboard/huboard-web/vender/gems/faye_extensions"
Copy link
Member

Choose a reason for hiding this comment

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

@dahlbyk
Copy link
Member

dahlbyk commented Feb 25, 2016

I'm going to pull this down with #201 and see if a clean vagrant up still shows the Bundler weirdness I was seeing before downgrading to 1.9.

s.homepage = "TODO"
s.summary = "TODO: Summary of Saas."
s.description = "TODO: Description of Saas."
s.homepage = "https://github.com/huboard/huboard-web/tree/master/vendor/gems/faye_extensions "
Copy link
Member

Choose a reason for hiding this comment

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

Extra space at the end :)

@discorick
Copy link
Member Author

Good call, if that fixes it ill throw my hat (I dont have a hat...)

@rauhryan
Copy link
Member

Looks pretty harmeless

:shipit: ?

@dahlbyk
Copy link
Member

dahlbyk commented Feb 25, 2016

I'm going to pull this down with #201 and see if a clean vagrant up still shows the Bundler weirdness I was seeing before downgrading to 1.9.

I don't remember 100% what I was seeing before, but everything seems to be 🆒 locally for me.

You might try a clean git clone and install of your own to confirm, but regardless I see no problem shipping this as-is even if it doesn't completely resolve issues with Bundler 1.10+. Merging.

dahlbyk added a commit that referenced this pull request Feb 25, 2016
…ation_warnings

Resolves gemspec deprecation warnings on the Rails engine and Faye Extensions
@dahlbyk dahlbyk merged commit 847042e into master Feb 25, 2016
@dahlbyk dahlbyk deleted the discorick/resolve_gemspec_deprecation_warnings branch February 25, 2016 23:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants