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

Update httpparty dependency #47

Closed

Conversation

surfacedamage
Copy link

@surfacedamage surfacedamage commented Feb 13, 2020

Background

The current version of ruby-pardot has a dependency on the 0.13.0 version of httpparty which requires an older version of the json gem. This can cause dependency incompatibilities when attempting to include the gem on projects that have a requirement of a newer version of the json gem.

Starting with version 0.14.0 of httpparty, the json dependency was removed completely. This PR updates httpparty to the latest version.

Detail

This PR does two things:

  1. Fix broken specs on master due to missing characters in the XML stubs in the http-spec
  2. Updates the httpparty gem dependency to the latest version

References

The missing `<` would cause specs to fail.
`httpparty` version 0.13.1 has a dependency on an
older version of the `json` gem.  This can cause
incompatibility issues on projects that include
gems requiring more recent versions of the `json`
gem.

Beginning with version 0.14.0 of `httpparty`, the
`json` dependency was removed completely.

Upgrading to the latest version of `httpparty`
made sense.
@ebinmore ebinmore mentioned this pull request Feb 24, 2020
@0xStarcat
Copy link

let's merge this!

@ghost
Copy link

ghost commented Feb 1, 2021

Looks like the httparty version was updated in 08/2020 with PR #51 and the http_spec was fixed in #53. I'll close this PR but feel free to open it again if something was missed. Sorry for the late reply on this PR.

@ghost ghost closed this Feb 1, 2021
This pull request was closed.
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.

2 participants