New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[type-declarations] Add ctor based property types in - Lead bundles #12952
[type-declarations] Add ctor based property types in - Lead bundles #12952
Conversation
bc05a13
to
f92d733
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## 5.x #12952 +/- ##
============================================
- Coverage 58.86% 58.86% -0.01%
Complexity 33578 33578
============================================
Files 2181 2181
Lines 101759 101759
============================================
- Hits 59899 59897 -2
- Misses 41860 41862 +2
|
7c2742e
to
596a464
Compare
2dc9e2b
to
9f41d54
Compare
Let's wait for other PRs to get merge first, as this will be a bigger rebase. |
9f41d54
to
69f2562
Compare
Ready to go ✔️ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't see any issues with these changes. 👍
@TomasVotruba I started to think that since Mautic 5 supports only PHP 8+, we could switch directly to constructor property promotion instead of defining properties on the class level, right? Isn't this a change that is already outdated?
@escopecz PHP 8.0 is great target 👍 I plan to add property promotion PRs after those typed property PRs are merged. It's more safe to do it step by step 🙏 |
Thank you 👍 |
No description provided.