In #711918: Documentation standard for @param and @return data types, @param data types are being added to the doxygen standards for D8.

Morbus suggested in #1360276-5: Add xjm as maintainer or co-maintainer? that rules for this standard should be added to Coder Review rather than this project. However, we should remove the existing rule and allow Coder Review to handle them as appropriate (required for D8, and possibly disallowed in D7).

Comments

xjm’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Issue summary: View changes

Updated issue summary.

xjm’s picture

Status: Active » Needs review
StatusFileSize
new1.16 KB

Attached patch simply removes this warning.

danepowell’s picture

+1... will test this out and report back

danepowell’s picture

StatusFileSize
new1.26 KB

Patch in #1 doesn't apply cleanly to HEAD- here's an updated patch.

danepowell’s picture

StatusFileSize
new1.35 KB

My bad... please ignore #3. The patch failed to apply due to a misconfiguration.

However, I found another very similar bug- 'object' is not recognized as a valid Doxygen return type. The attached patch adds 'object' to the valid return types.

Since technically any sort of class can now be defined as a return type, should this rule be changed in a more general way? Maybe to look for spaces that would indicate a description rather than a return type?

xjm’s picture

Re: #4: Please open a separate issue for the other bug. Thanks!

Status: Fixed » Closed (fixed)

Automatically closed -- issue fixed for 2 weeks with no activity.

Anonymous’s picture

Issue summary: View changes

Updated issue summary.