- Issue created by @dww
- First commit to issue fork.
- Merge request !5471Convert FormattableMarkup to strings (simple replacement) in core/tests/Drupal/KernelTests/* → (Closed) created by mstrelan
- Status changed to Needs review
10 months ago 12:58am 20 November 2023 - Status changed to Needs work
10 months ago 1:54am 26 November 2023 - 🇺🇸United States smustgrave
Agreed with both your open threads about variables for readability.
- First commit to issue fork.
- Status changed to Needs review
10 months ago 11:24am 27 November 2023 - 🇮🇳India ankithashetty Karnataka, India
Variables updated in MR. Please verify if that was the ask in MR comments, thanks!
- 🇦🇺Australia mstrelan
No I think the idea is to revert the change from this issue and apply it in 📌 Fix strict type errors: Convert FormattableMarkup to strings (complex replacement) in core Kernel tests Active instead. While it's not "complex", this issue is just for super simple conversions (no new variables).
- Status changed to Needs work
10 months ago 3:23pm 27 November 2023 - Status changed to Needs review
8 months ago 5:31am 9 January 2024 - Status changed to RTBC
8 months ago 2:55pm 9 January 2024 - 🇬🇧United Kingdom longwave UK
Backported to 10.2.x to keep tests in sync.
Committed and pushed 7835ce45f3 to 11.x and 90632d1697 to 10.2.x. Thanks!
-
longwave →
committed 90632d16 on 10.2.x
Issue #3402292 by mstrelan, ankithashetty, dww, smustgrave: Fix strict...
-
longwave →
committed 90632d16 on 10.2.x
- Status changed to Fixed
8 months ago 6:15pm 12 January 2024 -
longwave →
committed 7835ce45 on 11.x
Issue #3402292 by mstrelan, ankithashetty, dww, smustgrave: Fix strict...
-
longwave →
committed 7835ce45 on 11.x
Automatically closed - issue fixed for 2 weeks with no activity.