Skip to content

support faraday 2.x#27

Open
Kallin wants to merge 1 commit intoOneHQ:masterfrom
Kallin:faraday-2-support
Open

support faraday 2.x#27
Kallin wants to merge 1 commit intoOneHQ:masterfrom
Kallin:faraday-2-support

Conversation

@Kallin
Copy link

@Kallin Kallin commented Aug 16, 2024

Description

The current gemspec prevents this project from being used in projects that are using modern farday (2.x). This PR allows sendgrid_webapi to operate in either environment.

Checklist

  • I have added appropriate tests if this PR fixes a bug or adds a feature.
  • All status checks (tests, linting, etc...) are passing.

- modify cassettes to show correct content-type on response (required for faraday to auto-parse JSON)
@Kallin
Copy link
Author

Kallin commented Aug 16, 2024

not sure why specs are failing. I ran rake locally and tests pass.

@Kallin
Copy link
Author

Kallin commented Aug 16, 2024

@itmwiw , any chance you or another maintainer could take a look at this? 🙏

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.

1 participant