Problem/Motivation
In addition to what's in #3217315: Update non-dev dependencies to latest minor as of June 3, 2021 and #3217320: Update dependencies to latest patch-level releases as of June 3, 2021:
- composer/xdebug-handler released a new major (2) on April 9, 2021, and 2.0.1 on May 5, 2021.
- composer/composer released a new minor (2.1) on June 3, 2021.
It would be good to get these up to date. Since they are dev dependencies, maybe they are still eligible for Drupal 9.2?
Steps to reproduce
Proposed resolution
Create patch by first applying #3217315: Update non-dev dependencies to latest minor as of June 3, 2021 and #3217320: Update dependencies to latest patch-level releases as of June 3, 2021 and then running composer update --with phpunit/phpunit:^8
.
Upgrading to PHPUnit 9 should be done in a separate issue.
Comments
Comment #2
effulgentsia CreditAttribution: effulgentsia at Acquia commentedHere's a patch for just the scope of this issue (rolled on top of #3217315: Update non-dev dependencies to latest minor as of June 3, 2021 and #3217320: Update dependencies to latest patch-level releases as of June 3, 2021), and one that's combined with those two to get DrupalCI runs ahead of those ones landing.
Comment #3
effulgentsia CreditAttribution: effulgentsia at Acquia commentedOnce the other two land, the non-combined patch in #2 should apply to both 9.2.x and 9.3.x. However, per #3217320-4: Update dependencies to latest patch-level releases as of June 3, 2021, the combined patch doesn't apply on 9.3.x because of a context line difference. Here's a patch that fixes that, so that we can get a DrupalCI run on 9.3 in the meantime.
Comment #4
effulgentsia CreditAttribution: effulgentsia at Acquia commentedNote that https://github.com/cweagans/composer-patches/issues/363 reports some problems using Composer Patches on Composer 2.1, though I think that would only be relevant to the version of Composer being used on the CLI, which doesn't have to be the same as what Drupal core requires as a dev dependency.
Comment #5
xjmThis issue hadn't been marked RTBC yet, but the first one had. I don't think it makes sense to do this as separate commits in this case.
I'm going to go ahead and commit this, and then if folks have concerns we can discuss a revert.
Comment #7
xjmSaving credits from the three issues.
Comment #9
xjmComment #10
effulgentsia CreditAttribution: effulgentsia at Acquia commentedSmall followup: #3217447: Update patch-level dependencies for June 10, 2021. That's just a patch release to a dev dependency, so no urgency on it.
Comment #11
effulgentsia CreditAttribution: effulgentsia at Acquia commented#3217451: [TESTING ONLY; DO NOT COMMIT] Update to PHPUnit 9