Problem/Motivation

Prior to Drupal 9.2.0 we should update our dependencies to the latest minor releases where possible.

Steps to reproduce

Proposed resolution

Remaining tasks

User interface changes

API changes

Data model changes

Release notes snippet

CommentFileSizeAuthor
#6 interdiff.3206301.4-6.txt718 byteslongwave
#6 3206301-6.patch98.51 KBlongwave
#4 interdiff.3206301.2-4.txt928 byteslongwave
#4 3206301-4.patch98.51 KBlongwave
#2 3206301-2.patch97.6 KBlongwave
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

longwave created an issue. See original summary.

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Active » Needs review
FileSize
3206301-2.patch97.6 KB

Includes a PHPUnit fix from #3204947: Update PHPUnit to 8.5.15 plus the following version bumps:

+--------------------------------------+---------+---------+
| Production Changes                   | From    | To      |
+--------------------------------------+---------+---------+
| composer/installers                  | v1.9.0  | v1.10.0 |
| doctrine/annotations                 | 1.11.1  | 1.12.1  |
| egulias/email-validator              | 2.1.24  | 2.1.25  |
| guzzlehttp/promises                  | 1.4.0   | 1.4.1   |
| guzzlehttp/psr7                      | 1.7.0   | 1.8.1   |
| laminas/laminas-feed                 | 2.13.0  | 2.14.0  |
| laminas/laminas-zendframework-bridge | 1.1.1   | 1.2.0   |
| pear/pear_exception                  | v1.0.1  | v1.0.2  |
| psr/container                        | 1.0.0   | 1.1.1   |
| symfony/console                      | v4.4.17 | v4.4.21 |
| symfony/debug                        | v4.4.17 | v4.4.20 |
| symfony/dependency-injection         | v4.4.17 | v4.4.21 |
| symfony/error-handler                | v4.4.17 | v4.4.21 |
| symfony/event-dispatcher             | v4.4.17 | v4.4.20 |
| symfony/http-foundation              | v4.4.17 | v4.4.20 |
| symfony/http-kernel                  | v4.4.17 | v4.4.21 |
| symfony/mime                         | v5.2.0  | v5.2.6  |
| symfony/polyfill-ctype               | v1.20.0 | v1.22.1 |
| symfony/polyfill-iconv               | v1.20.0 | v1.22.1 |
| symfony/polyfill-intl-idn            | v1.20.0 | v1.22.1 |
| symfony/polyfill-intl-normalizer     | v1.20.0 | v1.22.1 |
| symfony/polyfill-mbstring            | v1.20.0 | v1.22.1 |
| symfony/polyfill-php80               | v1.20.0 | v1.22.1 |
| symfony/process                      | v4.4.17 | v4.4.20 |
| symfony/psr-http-message-bridge      | v2.0.2  | v2.1.0  |
| symfony/routing                      | v4.4.17 | v4.4.20 |
| symfony/serializer                   | v4.4.17 | v4.4.20 |
| symfony/translation                  | v4.4.17 | v4.4.21 |
| symfony/validator                    | v4.4.17 | v4.4.21 |
| symfony/var-dumper                   | v5.2.0  | v5.2.6  |
| symfony/yaml                         | v4.4.17 | v4.4.21 |
| twig/twig                            | v2.14.1 | v2.14.4 |
+--------------------------------------+---------+---------+

+------------------------------------+---------+---------+
| Dev Changes                        | From    | To      |
+------------------------------------+---------+---------+
| composer/ca-bundle                 | 1.2.8   | 1.2.9   |
| composer/composer                  | 2.0.8   | 2.0.11  |
| composer/xdebug-handler            | 1.4.5   | 1.4.6   |
| drupal/coder                       | 8.3.11  | 8.3.12  |
| phar-io/version                    | 3.0.3   | 3.1.0   |
| phpspec/prophecy                   | 1.12.1  | 1.13.0  |
| phpunit/phpunit                    | 8.5.13  | 8.5.15  |
| sirbrillig/phpcs-variable-analysis | v2.10.0 | v2.11.0 |
| symfony/browser-kit                | v4.4.17 | v4.4.20 |
| symfony/css-selector               | v4.4.17 | v4.4.20 |
| symfony/dom-crawler                | v4.4.17 | v4.4.20 |
| symfony/filesystem                 | v4.4.17 | v4.4.21 |
| symfony/finder                     | v4.4.17 | v4.4.20 |
| symfony/lock                       | v4.4.17 | v4.4.21 |
| symfony/phpunit-bridge             | v5.2.0  | v5.2.6  |
| webmozart/assert                   | 1.9.1   | 1.10.0  |
+------------------------------------+---------+---------+

Status: Needs review » Needs work

The last submitted patch, 2: 3206301-2.patch, failed testing. View results

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Needs work » Needs review
FileSize
3206301-4.patch98.51 KB
interdiff.3206301.2-4.txt928 bytes
renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Needs review » Needs work

In this case there is a comment before the deprecated action

// assertInternalType() can have many different messages.

But this comment is so generic, please can we put more information on this comment to be clear?

longwave’s picture

longwave
he/him
Primary language English
Location UK
CreditAttribution: longwave at Full Fat Things commented
Status: Needs work » Needs review
FileSize
3206301-6.patch98.51 KB
interdiff.3206301.4-6.txt718 bytes

I think this is a bit better?

renatog’s picture

renatog
Location Campinas
CreditAttribution: renatog at CI&T commented
Status: Needs review » Reviewed & tested by the community

That's perfect. Clear now
#6 works well, so moving to RTBC

  • catch committed 8be87ec on 9.2.x 
    Issue #3206301 by longwave, RenatoG: Update dependencies for Drupal 9.2
catch’s picture

catch
he/him
Primary language English
CreditAttribution: catch at Third and Grove commented
Status: Reviewed & tested by the community » Fixed

Committed/pushed to 9.2.x, thanks!

Status: Fixed » Closed (fixed)

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

effulgentsia’s picture

effulgentsia CreditAttribution: effulgentsia at Acquia commented
Issue tags: +9.2.0 release notes