-
-
Notifications
You must be signed in to change notification settings - Fork 5.2k
[Contributing] Fix Coding Standards example (esp. null return vs. void) #12974
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
Conversation
What's the matter with the "DOCtor-RST" lint failing? |
@guilliamxavier you can ignore the DOCtor-RST linter errors for this pull request. About the proposed changes, let's ask @nicolas-grekas and @stof for a review. Thanks! |
* @param bool $theSwitch Some switch to control the method's flow | ||
* | ||
* @return bool|void The resultant check if $theSwitch isn't false, void otherwise |
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.
Note: I have seen some @return void
in the code, but below is explicitly stated
Omit the
@return
tag if the method does not return anything
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.
Just one minor note and GTM
@nicolas-grekas: Thank you for reviewing, I have applied your suggested changes |
Thank you @guilliamxavier. |
Fix inconsistent return points and make the logic a bit more realistic (even if still dummy code)