docs(ast): enforce doc comments on AST node methods

[DonIsaac]

Part of oxc-project/backlog#130

[graphite-app]

Your org has enabled the Graphite merge queue for merging into main

Add the label “0-merge” to the PR and Graphite will automatically add it to the merge queue when it’s ready to merge. Or use the label “hotfix” to add to the merge queue as a hot fix.

You must have a Graphite account and log in to Graphite in order to use the merge queue. Sign up using this link.

[DonIsaac]

• docs(ast): enable crate-wide warnings on missing doc comments #6716
• docs(ast): enforce doc comments on AST node methods #6714 👈
• docs(ast): enforce doc comments on generated ASTBuilder methods #6713
• docs(ast): enforce doc comments on JSX nodes, literal nodes, and comments #6712
• docs(codegen): add #![warn(missing_docs)] to oxc_codegen #6711
• docs(macros): add #![warn(missing_docs] to oxc_macros #6710
• main

This stack of pull requests is managed by Graphite. Learn more about stacking.

Join @DonIsaac and the rest of your teammates on Graphite

[codspeed-hq]

CodSpeed Performance Report

Merging #6714 will not alter performance

_{Comparing don/10-20-docs_ast_enforce_doc_comments_on_ast_node_methods (a7dd5aa) with main (abb4ff3)}

Summary

✅ 30 untouched benchmarks

[Boshen]

Merge activity

• Oct 20, 10:00 PM EDT: The merge label '0-merge' was detected. This PR will be added to the Graphite merge queue once it meets the requirements.
• Oct 20, 10:20 PM EDT: A user added this pull request to the Graphite merge queue.
• Oct 20, 10:36 PM EDT: The Graphite merge queue removed this pull request due to downstack failures on PR #6713.
• Oct 20, 10:37 PM EDT: The Graphite merge queue removed this pull request due to downstack failures on PR #6713.
• Oct 21, 1:29 AM EDT: A user added this pull request to the Graphite merge queue.
• Oct 21, 1:35 AM EDT: A user merged this pull request with the Graphite merge queue.