- Issue created by @alexpott
- 🇬🇧United Kingdom alexpott 🇪🇺🌍
Made this a core-mvp to reduce any API change after the MR for core has been committed.
- First commit to issue fork.
- Status changed to Needs review
7 months ago 2:02am 23 April 2024 - 🇺🇸United States thejimbirch Cape Cod, Massachusetts
Went through the following directories and added
declare(strict_types=1);to any file that didn't have it.Drupal\Core\Config\Action
Drupal\Core\Config\Checkpoint
Drupal\Core\DefaultContent
Drupal\Core\Recipe - Status changed to RTBC
7 months ago 2:45am 23 April 2024 - 🇮🇳India mukesh-kumar3 Dharmshala
@thejimbirch I have verified this fix. Taken the pull from this issue branch and it added the declare(strict_types=1); in below.
Drupal\Core\Config\Action
Drupal\Core\Config\Checkpoint
Drupal\Core\DefaultContent
Drupal\Core\Recipe - 🇬🇧United Kingdom alexpott 🇪🇺🌍
Committed and pushed 5cacfa6f791 to 11.x and 32aa5d54c63 to 10.3.x. Thanks!
- Status changed to Fixed
7 months ago 8:57am 23 April 2024 - 🇬🇧United Kingdom alexpott 🇪🇺🌍
Cherry picked to ✨ Add recipes api as experimental API to core Needs review too.
-
alexpott →
committed 5cacfa6f on 11.x
Issue #3442304 by thejimbirch: Use strict types in all the new code
-
alexpott →
committed 5cacfa6f on 11.x
-
alexpott →
committed 32aa5d54 on 10.3.x
Issue #3442304 by thejimbirch: Use strict types in all the new code (...
-
alexpott →
committed 32aa5d54 on 10.3.x
- ffc594bd committed on patch
Update recipe 10.3.x patch 32aa5d54 Issue #3442304 by thejimbirch: Use...
- ffc594bd committed on patch
- b5870201 committed on patch
Update recipe 11.x patch 5cacfa6f Issue #3442304 by thejimbirch: Use...
- b5870201 committed on patch
Automatically closed - issue fixed for 2 weeks with no activity.