-
-
Notifications
You must be signed in to change notification settings - Fork 913
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
RFC: convert to use yard
for documentation
#1996
Comments
Note that one of the big reasons I'd like to do this is to start grouping methods in large classes like |
and do the same for Searchable related to #1996
Of course, I'm noticing now that RDoc supports much of what I wanted to use yard for, including:
So maybe this issue should be reviewed, and either closed or rewritten to be outcome-oriented (instead of solution-oriented). |
After thinking about the foundational nature of Nokogiri in the ecosystem, I think we should use RDoc to ensure that as many folks as possible are able to generate and read the doc strings. After reading docs, I think we can use RDoc without making it challenging for folks to navigate those docs, thanks to the |
Flip-flopping on this. I'm choosing YARD over RDoc/SDoc because:
|
Flip-flopping again, and investing in making RDoc better:
|
Note that RDoc's Markdown parser emits docs that don't look quite right throughout the RDoc toolchain (e.g., |
We're already using
yard
to generate the docs at https://nokogiri.org/rdoc/index.html, but we're not taking advantage of some of yard's features (like the ability to embed mixin methods, e.g.,Searchable
inNode
andNodeSet
), and we're certainly not using yard syntax (e.g., all thecall-seq
annotations that we currently use are ignored).So I'm thinking it would be best to convert the doc formatting to be yard-compatible markdown, and eliminate rdoc format entirely from the codebase.
I'd love to hear comments on this proposal.
Note to future self: try using https://github.com/postmodern/hoe-yard to drive some of this; and then have the
nokogiri.org
docs task just call nokogiri's own rake task for the site generation.The text was updated successfully, but these errors were encountered: