-
Notifications
You must be signed in to change notification settings - Fork 95
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
Fix static analysis issues #968
Conversation
…s not a valid value for display property
Kudos, SonarCloud Quality Gate passed! 0 Bugs No Coverage information |
Hm, not sure why that unit test failed. Can't tell if related to this change or not. |
Thanks @kinow! Looks good! I will test locally and if everything works fine after the changes, I will merge this. No need to squash - it's good to have small, descriptive commits. One of the Travis jobs failed, probably just some random transient issue. I restarted it. |
Codecov Report
@@ Coverage Diff @@
## master #968 +/- ##
============================================
- Coverage 58.80% 58.79% -0.02%
Complexity 1505 1505
============================================
Files 32 32
Lines 4207 4208 +1
============================================
Hits 2474 2474
- Misses 1733 1734 +1
Continue to review full report at Codecov.
|
Couldn't find any issues, and the Travis CI job passed on the second attempt. Thanks again! |
Left multiple commits for now in-case we need to drop some. But happy to squash (or OK if somebody else squashes them) commits. Had some time today while another thing was running, so went through the code, the static analysis issues reported in SonarCloud and PhpStorm and fixed a few ones.
Cheers
Bruno