Skip to content
This repository has been archived by the owner on Apr 22, 2023. It is now read-only.

build: use doc-tool module #9356

Closed
wants to merge 1 commit into from

Conversation

robertkowalski
Copy link

delete old checked in doc-tool. run npm i -g node-doc-generator
before building the docs.

all the best from newark :)

delete old checked in doc-tool. run `npm i -g node-doc-generator`
before building the docs.
@robertkowalski
Copy link
Author

@misterdjules i don't remember how you double checked if something for the doc changed exactly (it was a rsync dry run) can you send me a hint - so far i just verified by looking at the results and by writing the tests

@misterdjules
Copy link

@robertkowalski I use rsync -v -n -c -u -r out/doc/ node@nodejs.org:~/web/nodejs.org/ to check what changes would be deployed after making changes to the API docs.

@misterdjules
Copy link

Same comments as in nodejs/nodejs.org-archive#84:

Instead of invoking a global node-doc-generator, would it be better to install it locally and update/install it every time the documentation is generated so that we don't run the risk of generating documentation with an outdated generator?

@jasnell
Copy link
Member

jasnell commented Aug 15, 2015

@robertkowalski @misterdjules ... any reason to keep this open here? can we move to one of the other repos?

@robertkowalski
Copy link
Author

good point, closing

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants