When I use composer to try and install Drush 13 I get this error:

- drupal/warmer dev-2.x requires drush/drush ^11.6 || ^12 -> found drush/drush[11.6.0, 12.0.0, ..., 12.5.3] but it conflicts with your root composer.json require (^13).

My reason for upgrading is to get my environment ready for 11.

Upgrade report

Drush 13 also runs on 10.2 or later.

Drush compatibility

Is this a quick fix of warmer/composer.json or is more involved?

CommentFileSizeAuthor
#3 warmer-3467609.patch693 bytesrajan kumar@2026
Screenshot 2024-08-12 at 9.03.24 AM.png191.59 KBduckydan
Screenshot 2024-08-12 at 9.04.06 AM.png57.68 KBduckydan

Issue fork warmer-3467609

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

duckydan created an issue. See original summary.

andypost’s picture

andypost
he/him
Primary language Russian
CreditAttribution: andypost as a volunteer and at Skilld commented
Related issues: +#3435591: Automated Drupal 11 compatibility fixes for warmer
rajan kumar@2026’s picture

rajan kumar@2026 CreditAttribution: rajan kumar@2026 commented
Status: Active » Needs review
StatusFileSize
new warmer-3467609.patch693 bytes

Hi @duckydan

I have fixed the issue and created a patch. Thank you.

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

abhinesh opened merge request !17

bhaveshdas’s picture

bhaveshdas CreditAttribution: bhaveshdas as a volunteer and at cmsMinds for Drupal India Association commented
Assigned: Unassigned » bhaveshdas
bhaveshdas’s picture

bhaveshdas CreditAttribution: bhaveshdas as a volunteer and at cmsMinds for Drupal India Association commented
Assigned: bhaveshdas » Unassigned
duckydan’s picture

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

Thanks! It makes the change in the local module composer file.

hudri’s picture

hudri
Primary language German
Location Austria
CreditAttribution: hudri at webtourismus.at commented

Do we need the hard Drush dependency at all in the composer.json file? It is an optional CLI command, not a functional requirement. The module does it's job great via cron and UI.

I'm proposing to completely remove the drush dependency from composer.json.

jigarius’s picture

jigarius
Location Montréal
CreditAttribution: jigarius as a volunteer and at Symetris commented

I agree with hudri. We should remove the drush/drush dependency.

jigarius opened merge request !20

jigarius’s picture

jigarius
Location Montréal
CreditAttribution: jigarius as a volunteer and at Symetris commented
Status: Reviewed & tested by the community » Needs review
duckydan’s picture

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

Works great.

maxilein’s picture

maxilein CreditAttribution: maxilein commented

It did not work for me.
D10.3.6 php 8.3 drush 13.2

maxilein’s picture

maxilein CreditAttribution: maxilein commented

Just installed the current dev on D11.1.
Works fine with drush 13 even without this patch.

duckydan’s picture

duckydan CreditAttribution: duckydan commented
Status: Reviewed & tested by the community » Fixed

I think it was fixed here: https://git.drupalcode.org/project/warmer/-/commit/72d7dab49b120d1a3c886...

I’ll mark this as fixed.

Thanks for pointing it out.

Status: Fixed » Closed (fixed)

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