- Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7last update
9 months ago Not currently mergeable. - @bobi-mel opened merge request.
- last update
9 months ago 2 fail - Issue was unassigned.
- Status changed to Needs review
9 months ago 2:44pm 5 October 2023 - 🇺🇦Ukraine Bobik
The type hint in function parameters and return values was added by me. Please check it.
- Open on Drupal.org →Core: 10.1.x + Environment: PHP 8.1 & MySQL 5.7
24:26 24:26 Queueing - last update
8 months ago 2 fail - First commit to issue fork.
- Status changed to Needs work
8 months ago 6:26am 21 October 2023 - 🇬🇧United Kingdom scott_euser
At a glance I cannot see why this is having such an effect causing those two tests to fail. Needs further investigation
- Assigned to Bobik
- Issue was unassigned.
- Status changed to Fixed
8 months ago 5:12am 26 October 2023 - 🇬🇧United Kingdom scott_euser
I have merged this into the more significant work in progress happening here https://www.drupal.org/project/site_settings/issues/3395787 ✨ Provide new site settings loader that uses manage display, etc rather than flattening Needs work to avoid it becoming too hard to merge. Will sort the test issue there if it still occurs.
Thanks!
-
bobi-mel →
committed 1efce5a2 on 2.0.x
Issue #3176342: Add type hint in function parameters and return values
-
bobi-mel →
committed 1efce5a2 on 2.0.x
-
bobi-mel →
committed b23b84f7 on 2.0.x
Issue #3176342: Add type hint in function parameters and return values
-
bobi-mel →
committed b23b84f7 on 2.0.x
Automatically closed - issue fixed for 2 weeks with no activity.