-
Notifications
You must be signed in to change notification settings - Fork 5
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
Issue 259 #261
Conversation
Qodana for PHP80 new problems were found
💡 Qodana analysis was run in the pull request mode: only the changed files were checked Contact Qodana teamContact us at [email protected]
|
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.
My review covers only code/logic, no psalm/phpcs/readability related observations.
I will provide that level of review once the PR is in it's final stage.
@MarioRadu add your review to this PR, in case that I missed something |
Initial commit of this issue is for review purpose, i need to make the tests for this