- Issue created by @mondrake
- last update
over 1 year ago 29,597 pass, 318 fail - @mondrake opened merge request.
- last update
over 1 year ago 30,080 pass, 32 fail - last update
over 1 year ago Custom Commands Failed - last update
over 1 year ago 30,168 pass - Status changed to Needs review
over 1 year ago 10:06am 20 September 2023 - Status changed to RTBC
over 1 year ago 2:16pm 20 September 2023 - 🇺🇸United States smustgrave
Going to mark as the deprecation seems good.
Only question is if we can reuse that CR or need a new one. That I can't answer.
- last update
about 1 year ago 30,168 pass - last update
about 1 year ago 30,205 pass - last update
about 1 year ago 30,363 pass - last update
about 1 year ago 30,365 pass - last update
about 1 year ago 30,360 pass - 🇺🇸United States xjm
Can we please attach the existing CR to this issue? Edit: Found it and did so, including adding a "Drupal 10.2 update" that it will now properly raise a deprecation.
The diff itself looks great.
- 🇺🇸United States xjm
Saving credits. @smustgrave for the question about the CR and me for fixing it. :P
- Status changed to Fixed
about 1 year ago 7:23pm 30 September 2023 - 🇺🇸United States xjm
Committed to 11.x. Not backporting since it includes a new deprecation warning. Thanks!
I also toggled the status to "Draft" and back so that the CR would show up as a recent one instead of being buried in 2014 when Drupal 8.0.0 was still only an alpha.
- 🇮🇹Italy mondrake 🇮🇹
Filed 🐛 MergeTest::tesDeprecatedKeyArrayArgument() is mispelled Active , the test method name is misspelled.
Automatically closed - issue fixed for 2 weeks with no activity.