Uh oh!
There was an error while loading. Please reload this page.
doc: add in guide generation using remark - #5408
Conversation
*Do not merge.* I did not commit in node_modules. This intends to generate guides, it is related to #4866. This generates HTML from MD and implements highlight.js.
| mdast = processor.run(mdast); | ||
| const html = processor.stringify(mdast, remarkOptions); | ||
| // Locate YAML front matter | ||
| if (mdast.children[0] && mdast.children[0].type === 'yaml') { |
There was a problem hiding this comment.
Why not use a YAML parser such as js-yaml? Currently this code wouldn't support the full YAML specification, which could be confusing for people expecting to be able to write true YAML within the docs.
silverwind
commented
Feb 25, 2016
Regarding node_modules, I'd suggest
Of course, the less dependencies and files, the better :) |
silverwind
commented
Feb 25, 2016
Of course, we could also go the route of requiring the user to install the dependencies if these guides are not part of the release build. |
Fishrock123
commented
Feb 25, 2016
I'm inclined to prefer not having them checked in, so long as the website build is able to work with that. |
Qard
commented
Feb 25, 2016
I think we can actually leave YAML parsing out of this entirely for now. It's mainly for eventually replacing the reference doctool that we want YAML frontmatter for. We can get to that later though. For now, we just want something that can build only the guide files in a way that it can be delivered with each release build to the website. |
jasnell
commented
Mar 21, 2016
What's the status on this one? |
Qard
commented
Mar 22, 2016
@kahwee Are you able to continue working on this? I can volunteer some time next week to help get this ready to merge, if you aren't able to get it finished up yourself. |
7da4fd4 to
c7066fbCompareeljefedelrodeodeljefe
commented
May 2, 2016
@Qard will you find some time to pick this up? I guess it's blocking a lot of new doc stuff. Also we would need to continue discussion where guids will go on the website afterwards, right? cc @nodejs/documentation |
Qard
commented
May 2, 2016
I'll see what I can do. I don't have a ton of time available at the moment though, so I'd appreciate any help I can get. If other docs people could review this PR and see if they can put together any of the other modules we might need, that'd be a big help. |
eljefedelrodeodeljefe
commented
May 2, 2016
Alright. No pressure. I think I got some time tomorrow. |
eljefedelrodeodeljefe
commented
May 25, 2016
Okay I have picked this up.
|
Do not merge.
I did not commit in node_modules. This intends to generate guides, it is related to #4866. This generates HTML from MarkDown and implements highlight.js.
Currently it generates
building-node-with-ninja.mdinto corresponding HTML.To run it:
This differs from #4866 by being group processing Markdown guide files.
CC: @nodejs/documentation
By the way, if I commit in
node_modules, there are a lot of files going in. Is this an approach we should proceed with?