-
Notifications
You must be signed in to change notification settings - Fork 31.2k
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 90.23% 90.24% +0.01%
==========================================
Files 630 630
Lines 185129 185129
Branches 36234 36238 +4
==========================================
+ Hits 167049 167075 +26
+ Misses 11037 11030 -7
+ Partials 7043 7024 -19 🚀 New features to boost your workflow:
|
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? |
aa7ca6b
to
65cad89
Compare
I've rebased, sorry for the delay. However, I don't this to interfere with #57343, and that should take prriority. |
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.