-
Notifications
You must be signed in to change notification settings - Fork 30.6k
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
cli: generate man-page #55885
base: main
Are you sure you want to change the base?
cli: generate man-page #55885
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #55885 +/- ##
==========================================
- Coverage 88.42% 87.92% -0.51%
==========================================
Files 654 654
Lines 187852 187852
Branches 36134 35830 -304
==========================================
- Hits 166102 165162 -940
- Misses 14989 15881 +892
- Partials 6761 6809 +48 |
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.
LGTM
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.
Do we need the lint-md to run on our generated stuff?
cc @avivkeller can you rebase? |
The tree of #55268 got messed up (sorry!), and I had to recreate.
CC @aduh95 @ovflowd (previous reviewers)
This PR adds generation of the
node.1
file to theformat-md
command. This way, the manpage is always up-to-date with a new release, as it currently contains outdated (and/or incorrect) information.This is done by using the
@node-core/api-docs-tooling
tool. Currently, it is locked to nodejs/api-docs-tooling@bbf44c4 to prevent any future changes from breaking this.