Problem/Motivation

Pointed out by @rkoller

the composer.json got successfully upgraded to "drupal/core-recommended": "9.2.5" but when i run ddev composer outdated --direct drupal/core-composer-scaffold and drupal/core-project-message are still at 9.2.4 . so the version gap might increase over the months for these two dependencies. wouldn't it be reasonable to upgrade those as well alongside to avoid possible conflicts in the future?

Probably a good idea.

Proposed resolution

Update these too

Issue fork automatic_updates-3233138

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
Support from Acquia helps fund testing for Drupal Acquia logo

Comments

tedbow created an issue. See original summary.

tedbow credited rkoller.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
CreditAttribution: tedbow at Acquia commented

phenaproxima made their first commit to this issue’s fork.

phenaproxima opened merge request !60

phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Active » Needs review
rkoller’s picture

rkoller
he/him
Primary language German
Location Nürnberg, Germany
CreditAttribution: rkoller commented
Status: Needs review » Reviewed & tested by the community

I've installed Drupal 9.2.4 as well as the latest dev version of automatic updates. I've then applied the .diff and and enabled the module afterwards. After checking the update state and then running the cron Drupal got updated to version 9.2.6. The versions in the composer.json file got updated accordingly: 

"drupal/core-composer-scaffold": "9.2.6",
"drupal/core-project-message": "9.2.6",
"drupal/core-recommended": "9.2.6"
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Reviewed & tested by the community » Needs review

Thanks for the test, @rkoller! I’m setting this back to review so I can get validation from @tedbow on the code and general approach before we commit it.

tedbow’s picture

tedbow
he/him
Primary language English
Location Ithaca, NY, USA
CreditAttribution: tedbow at Acquia commented
Status: Needs review » Reviewed & tested by the community

Looks good!

  • phenaproxima committed 2a30540 on 8.x-2.x 
    Issue #3233138 by phenaproxima, tedbow, rkoller: If necessary update...
phenaproxima’s picture

phenaproxima
he/him
Primary language English
Location Massachusetts
CreditAttribution: phenaproxima at Acquia commented
Status: Reviewed & tested by the community » Fixed

Status: Fixed » Closed (fixed)

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