Skip to content

Update to Gems#51

Open
collabital wants to merge 6 commits intoyammer:masterfrom
collabital:master
Open

Update to Gems#51
collabital wants to merge 6 commits intoyammer:masterfrom
collabital:master

Conversation

@collabital
Copy link

Updates to Rest-Client 2.0, in particular the way redirections are handled.

@yammer yammer deleted a comment from msftclas Sep 27, 2017
Copy link

@robertIngrum robertIngrum left a comment

Choose a reason for hiding this comment

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

Would be really nice to get this merged, having to deal with rest-client < 2.0 is a huge pain.

@robertIngrum
Copy link

@tiabas would it be possible to get this merged? Looking at the failed test it seems like there might be an issue with the CI rather than this branch.

@collabital
Copy link
Author

@robertIngrum I'd say the issue is the gem requiring Travis to test against Ruby 1.9.2. With OpenSSL 1.1 being certainly outdated, so has this version of Ruby. Has the time come to drop support for outdated Ruby versions?

@robertIngrum
Copy link

@collabital maybe, but I'm pretty sure this repo is long dead. We just ended up cutting a fork off this pr and thankfully haven't had to touch this gem since (that I know of).

@tiabas
Copy link
Contributor

tiabas commented Mar 16, 2020

Hey @robertIngrum I can help with this, I think. If someone can get the tests to pass. I can push out a new version of the gem

@synth
Copy link

synth commented Nov 29, 2022

A new version of the gem would be amaaazing!

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.

5 participants

Comments