- Issue created by @jurgenhaas
- @jurgenhaas opened merge request.
- 🇩🇪Germany jurgenhaas Gottmadingen
The MR307 declares PHP 8.1 as a minimum requirement and updates the most obvious code places to be more strict - but now no longer compatible with PHP 7.
Other code style improvements may follow later in separate issues to address other areas of code improvements WRT PHP 8.1, e.g. we have around a dozen places marked with
@phpstan-ignore-next-line, especially for cases where we dynamically declare object properties, which we need to replace eventually. - Issue was unassigned.
- Status changed to Needs review
almost 2 years ago 1:25pm 12 February 2023 - Status changed to RTBC
almost 2 years ago 1:46pm 13 February 2023 -
jurgenhaas →
committed 8d421bf1 on 1.2.x
Issue #3339571 by jurgenhaas, danielspeicher: Set PHP 8.1 as minimum...
-
jurgenhaas →
committed 8d421bf1 on 1.2.x
- Status changed to Fixed
almost 2 years ago 1:17pm 15 February 2023 - 🇩🇪Germany jurgenhaas Gottmadingen
Perfect, this is now our new base for the next phase of developments ;-)
- 🇩🇪Germany mxh Offenburg
According discussion about breaking API changes due to this change can be found here: https://drupal.slack.com/archives/C0287U62CSG/p1676478385894969
- 🇩🇪Germany mxh Offenburg
According discussion linked in #8 can also be found within section 4 of the linked issue.
Automatically closed - issue fixed for 2 weeks with no activity.
