rdevguide icon indicating copy to clipboard operation
rdevguide copied to clipboard

Mention tools::checkRd in documentation chapter

Open hturner opened this issue 1 year ago • 4 comments

hturner avatar Jan 29 '24 11:01 hturner

Add this function to documentation chapter -it picks up missing brackets, non ASCII characters (share examples about it - does not work vs works)

SaranjeetKaur avatar Dec 02 '24 11:12 SaranjeetKaur

@EllaKaye @SaranjeetKaur @hturner could you please review that pull, and reply if any changes..

7se7en72025 avatar Oct 08 '25 13:10 7se7en72025

Aside: I just wrote some code consuming the output of tools::checkRd() and was puzzled to learn that it just returns a classed string like checkRd: (%d) %s.

It seems to me that a data structure like list(line_number, finding) would be more useful instead of relying on string parsing.

We can see that print.checkRd() does just such string parsing:

https://github.com/r-devel/r-svn/blob/a2fb5899d5d7a8d21dfbfb4f710f70672784e672/src/library/tools/R/RdConv2.R#L1160-L1169

(writing here as a little more than a note-to-self prior to making it an r-devel post)

MichaelChirico avatar Oct 08 '25 16:10 MichaelChirico

Thank you @MichaelChirico sir, for your feedback and for confirming that my observation is valid. I appreciate your suggestion to post this on the r-devel mailing list, and I’ll share my idea there for broader discussion. If you have any tips for framing the post or additional context I should include, please let me know. Thanks again!

7se7en72025 avatar Oct 08 '25 19:10 7se7en72025