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

Loosen restriction on guzzle to allow for 7.x #41

Merged
merged 1 commit into from
Jun 8, 2022

Conversation

japerry
Copy link
Contributor

@japerry japerry commented Jun 6, 2022

When working on d10 readiness, we ran into some packages, including the sendgrid drupal module failing due to Guzzle 7.x being required in php 8.1 and D10.

When working on d10 readiness, we ran into some packages, including the sendgrid drupal module failing due to Guzzle 7.x being required in php 8.1 and D10.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 6, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mglaman
Copy link

mglaman commented Jun 6, 2022

The dep tree:

|--fastglass/sendgrid 2.0.4 (requires guzzlehttp/guzzle ^6.3)
|  |--drupal/sendgrid_integration 2.x-dev (requires fastglass/sendgrid ~2.0.0)

When running composer why-not guzzlehttp/guzzle ^7.0 -r -t against a D10 readiness dev site

@taz77
Copy link
Owner

taz77 commented Jun 8, 2022

👍 Thanks for the PR!

@taz77 taz77 merged commit fe3697e into taz77:master Jun 8, 2022
@taz77
Copy link
Owner

taz77 commented Jun 8, 2022

I tagged a new release too

@mglaman
Copy link

mglaman commented Jun 8, 2022

Thanks @taz77! Really appreciate it

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