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

Build Fix #235

Merged
merged 1 commit into from
Aug 8, 2018
Merged

Build Fix #235

merged 1 commit into from
Aug 8, 2018

Conversation

sahild
Copy link
Contributor

@sahild sahild commented Aug 10, 2014

This fixes the current build by stubbing few api's

This fixes the build by stubbing few api's
@sahild
Copy link
Contributor Author

sahild commented Aug 10, 2014

@hexgnu
Hey man nice gem , but failing build is kind of scary.
Can you merge this pull request after review
thanks

@hexgnu
Copy link
Owner

hexgnu commented Aug 12, 2014

Yea it's cause I changed mashie's version. I'll take a look sometime tomorrow / thursday to figure out what is going on since that change.

For now I think it's safer to use the pre 1.0.x version of linkedin.

@hexgnu
Copy link
Owner

hexgnu commented Aug 12, 2014

And by mashie I mean hashie :) I'll get to it, usually we keep a pretty clean build so don't fret too much.

@hundredwatt
Copy link
Collaborator

This fixes the cases broken by #224. Also addressed in #248. Safe to merge

@ryanatwork ryanatwork merged commit ac0383d into hexgnu:master Aug 8, 2018
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.

4 participants