Drupal 10 compatibility for 2.1.x-dev

Hello team!
Is this module finally going to be compatible with Drupal 10?

Thank you!

+1

Could this be task for 2.0 only to get beta 2 out with D10 support? I've renamed the title.

Adding patch.

Fixed tests (tested on 9.5):

Upgrade Status module does not complain anymore:

We are using this patch with Flystem S3 module and all seems good.

Could we push towards 2.0.0-beta2?

+1

After May 26th, 2.1.x has been released with a D10 compatibility so I think we should focus the effort to 2.1.x branch.

I have tested 2.1.0-rc4 with D10.0 and I got the following error:

Error: Call to undefined function Drupal\flysystem\Asset\file_create_url() in Drupal\flysystem\Asset\CssOptimizer->rewriteFileURI() (line 28 of modules/contrib/flysystem/src/Asset/CssOptimizer.php).

I could see that the patch attached here fix that line but it needs a reroll.

bbombachini → closed merge request !24

Sorry for all the issues with the fork, I was trying to change the target branch but I couldn't so I updated the existing fork and created a new branch on it.

This is working for me so I had removed the reroll tag and I'm marking as tested.

I've tested that the MR 25 fixes the last 10 issue.

I've tested MR 25 too and it fixes the last D10 issue

Thank you all for your contributions! This is merged into the dev branch, pending review of other outstanding issues, we'll try to get a new RC release shortly!