-
-
Notifications
You must be signed in to change notification settings - Fork 2.2k
PHP 7.4 syntax #12891
Copy link
Copy link
Closed
Labels
DXIssues and PRs aimed at improving Developer eXperience.Issues and PRs aimed at improving Developer eXperience.EnhancementMinor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).
Metadata
Metadata
Assignees
Labels
DXIssues and PRs aimed at improving Developer eXperience.Issues and PRs aimed at improving Developer eXperience.EnhancementMinor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).Minor issues and PRs improving the current solutions (optimizations, typo fixes, etc.).
This issue is created to coordinate efforts and have one space to reference to all of the PR's aiming to use PHP7.4 syntax in Sylius
Hello, folks! 👋
I've decided to open this issue, to be transparent with our efforts to finally use PHP 7.4 syntax on all Sylius' supported repositories. Let's face the truth - it's a little bit late 💃 But, on the other hand, better late than never 🐎
I already started the work, with this PR on SyliusResourceBundle. The path I chose is to use rectorphp, as it's working spectacularly with the code syntax upgrade. There are some things needed to be done additionally (probably upgrading PHPStan or other coding-standard relate libraries), but it should not be a big problem.
Feel free to participate, if you like to 🎉 I'm also sure, that we can do better (and faster) when we decide to bump the minimal PHP version to 8.0 (active support of PHP 7.4 ends at the end of November⚠️ ) 💪
To-do:
Live long and prosper, Sylius Community 🖖 🚀