-
-
Notifications
You must be signed in to change notification settings - Fork 853
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
Remove return value documentation for void return types. #1915
Conversation
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.
Tnx for fixing this for us.
Needs to be fixed at the template level. |
Should I change it only in the template or in both files? |
Both files should be changed, I completely missed that. |
3be054b
to
ba73add
Compare
The patch now includes the fix in the template as well. |
Codecov Report
@@ Coverage Diff @@
## master #1915 +/- ##
======================================
Coverage 87% 87%
======================================
Files 961 961
Lines 51034 51034
Branches 6324 6324
======================================
+ Hits 44789 44793 +4
+ Misses 5204 5202 -2
+ Partials 1041 1039 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Prerequisites
Description
Very minor code documentation fixes.