Problem/Motivation

In #3612544: Update to Guzzle 8 we added Guzzle 8 support while still allowing Guzzle 7, but I'm not sure the code changes actually support Guzzle 7.

Additionally, there are other dependencies where we currently allow multiple major versions, but for Drupal 12 let's start with a clean slate:

        "doctrine/lexer": "^2 || ^3",
        "guzzlehttp/guzzle": "^7.10 || ^8.0",
        "guzzlehttp/psr7": "^2.8.0 || ^3.0",
        "egulias/email-validator": "^3.2.1|^4.0",
        "sebastian/diff": "^4 || ^5 || ^6 || ^7",
        "justinrainbow/json-schema": "^5.2 || ^6.5.2",

Steps to reproduce

Proposed resolution

Drop support for the older major versions in Drupal 12.

Remaining tasks

User interface changes

Introduced terminology

API changes

Data model changes

Release notes snippet

Issue fork drupal-3612963

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Title: Drop Guzzle 7 support in Drupal 12 » Drop older dependency versions in Drupal 12
Issue summary: View changes

In fact let's reframe this a bit as there are some other dependencies where we technically allow multiple versions, but we should get a clean break in D12.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Status: Active » Needs review

longwave opened merge request !16428

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Issue summary: View changes
smustgrave’s picture

smustgrave commented
Status: Needs review » Reviewed & tested by the community

Lets do it!

  • larowlan committed 87de7098 on main 
    task: #3612963 Drop older dependency versions in Drupal 12

By: longwave
larowlan’s picture

larowlan
Location 🇦🇺🏝.au GMT+10
commented
Status: Reviewed & tested by the community » Fixed

Committed and pushed 87de7098664 to main. Thanks!

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented
Issue tags: +12.0.0 release notes
mradcliffe’s picture

mradcliffe
he/him
Primary language English
commented
Issue tags: +Needs change record

I think this needs a change record because we previously "supported" Guzzle 7 so developers and maintainers should know that they cannot make their modules backwards-compatible if they use Guzzle.

longwave’s picture

longwave
he/him
Primary language English
Location UK
commented

This has always been the way in major versions, and Guzzle 8 is largely backward compatible from what I have seen - all they have done is raise minimum PHP version and versions of some of their dependencies.

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.