- Issue created by @bbrala
- 🇭🇺Hungary Gábor Hojtsy Hungary
Gábor Hojtsy → made their first commit to this issue’s fork.
- 🇭🇺Hungary Gábor Hojtsy Hungary
Surprised its green, I would not believe it :D I did not update tests.
- Merge request !19Update bot code for Drupal 11 and PHP 8.1. Fixed some misc code style as I went along. → (Open) created by Gábor Hojtsy
- 🇳🇱Netherlands bbrala Netherlands
Thank you so much for working on this.
Seems this is pretty close. In order to get things running i did need to update some packages. Tie ran out, but will fully test. Been testing on jsonapi_extras.
I'm a little worries about double issues, which seemed to happen at some point, so want to make sure that is fixed.
Pushed my WIP changes to 3413931-fixed-d11-wip-changes
- 🇳🇱Netherlands bbrala Netherlands
Also, dryRun breaks the sql database for me locally, which took me a while to remember... fun :)
- 🇳🇱Netherlands bbrala Netherlands
Looks like its all good. It found existing issue (with patch) and then decided it was fine.
One thing we might need to do it give some sort of heads up that fixes provided by rector for Drupal 10 will require Drupal 10.1 and up. Although if composer/info.yml changes will be posted that will be kinda obvious.
- Status changed to RTBC
10 months ago 1:55pm 10 February 2024 - Status changed to Fixed
10 months ago 4:42pm 2 March 2024 Automatically closed - issue fixed for 2 weeks with no activity.