- Issue created by @project update bot
- last update
over 1 year ago 6 pass This is an automated patch generated by Drupal Rector. Please see the issue summary for more details.
It is important that any automated tests available are run with this patch and that you manually test this patch.
Drupal 10 Compatibility
According to the Upgrade Status module β , even with this patch, this module is not yet compatible with Drupal 10.
Currently Drupal Rector, version 0.15.1, cannot fix all Drupal 10 compatibility problems.
Therefore this patch does not update the
info.yml
file for Drupal 10 compatibility.Leaving this issue open, even after committing the current patch, will allow the Project Update Bot β to post additional Drupal 10 compatibility fixes as they become available in Drupal Rector.
Debug info
Bot run #12554This patch was created using these packages:
- mglaman/phpstan-drupal: 1.1.35
- palantirnet/drupal-rector: 0.15.1
- Assigned to Kristen Pol
- πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Assigning to myself to check against what's already been merged.
- Status changed to Needs work
over 1 year ago 7:59pm 15 August 2023 - πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
This is the change record: https://www.drupal.org/node/2907725 β
I looked and some of the code wasn't changed that should have been so marking as needs work.
Kristens-MacBook-Pro:fast_404 kristenpol$ grep -r getMockBuilder . ./tests/src/Unit/Fast404Test.php: $fast404 = $this->getMockBuilder('\Drupal\fast404\Fast404') ./tests/src/Unit/Fast404EventSubscriberTest.php: $fast404FactoryStackStub = $this->getMockBuilder('\Drupal\fast404\Fast404Factory') ./tests/src/Unit/Fast404EventSubscriberTest.php: $subscriber = $this->getMockBuilder('\Drupal\fast404\EventSubscriber\Fast404EventSubscriber')
- Status changed to Needs review
over 1 year ago 10:28pm 15 August 2023 - πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
I thought it was a rector issue so opened an issue:
π Rector rule for updating getMockBuilder is not catching all instances Closed: works as designed
but then I see those other places are chaining a method (
setConstructorArgs
) that can't be thrown out, so I closed that issue.This one should be ready to merge but I'll doublecheck by running some extra automated. tests on the patch.
- last update
over 1 year ago 6 pass - last update
over 1 year ago 6 pass - Status changed to RTBC
over 1 year ago 10:43pm 15 August 2023 - πΊπΈUnited States Kristen Pol Santa Cruz, CA, USA
Automated tests passed so this is ready to merge.
-
Kristen Pol β
committed b8a8d540 on 8.x-3.x
Issue #3369405 by Project Update Bot, Kristen Pol: Automated Drupal 10...
-
Kristen Pol β
committed b8a8d540 on 8.x-3.x
- Status changed to Fixed
11 months ago 7:41pm 6 February 2024 Automatically closed - issue fixed for 2 weeks with no activity.