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
[PHP 8.0] Add promoted properties in Stage, Sms, Report and Point bundles - step 3 #13005
[PHP 8.0] Add promoted properties in Stage, Sms, Report and Point bundles - step 3 #13005
Conversation
4ba2a95
to
bc6830d
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## 5.x #13005 +/- ##
============================================
- Coverage 58.85% 58.43% -0.43%
+ Complexity 33582 33577 -5
============================================
Files 2181 2181
Lines 101749 100480 -1269
============================================
- Hits 59887 58711 -1176
+ Misses 41862 41769 -93
|
e891abf
to
2aa7dfe
Compare
Good to go 👍 |
@@ -6,6 +6,7 @@ parameters: | |||
tmpDir: ./var/cache/phpstan | |||
level: 6 | |||
reportUnmatchedIgnoredErrors: false | |||
checkGenericClassInNonGenericObjectType: false |
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.
Is this disabled only because of this one issue? https://github.com/mautic/mautic/actions/runs/7125580913/job/19401784994#step:10:87
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.
Indeed 👍 personally I disable this until the full type coverage is reached, as it lead to focus on too detailed tiny spot and doesn't bring any value
Co-authored-by: John Linhart <jan@linhart.email>
Woop woop 👍 |
No description provided.