CI-lite: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/404/

Not sure what is wrong with CI. Is it infrastructural fails?

+ sed '/^/\s*$/d'
sed: 1: "/^/\s*$/d": invalid command code \

Somebody either changed the version of sed or changed the build script. So, yes, I would say this looks like infrastructure fails.

Between the two, I would say that changing the build script seems considerably more likely. The second slash just looks outright wrong. Perhaps they meant to have a backslash escaping the next backslash so that they shell would pass sed a regular expression containing only a single backslash. If that is indeed the problem, it would seem amusing given the change that this particular pull request is addressing. :-)

Replay: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/408/

Looks to be a real failure:

17:58:34 /data/iojs/build/workspace/node-test-commit-linuxone/nodes/rhel72-s390x/tools/doc/html.js:203
17:58:34             position
17:58:34             ^^^^^^^^
17:58:34 
17:58:34 SyntaxError: Unexpected identifier
17:58:34     at new Script (vm.js:72:7)
17:58:34     at createScript (vm.js:234:10)
17:58:34     at Object.runInThisContext (vm.js:286:10)
17:58:34     at Module._compile (internal/modules/cjs/loader.js:657:28)
17:58:34     at Object.Module._extensions..js (internal/modules/cjs/loader.js:700:10)
17:58:34     at Module.load (internal/modules/cjs/loader.js:599:32)
17:58:34     at tryModuleLoad (internal/modules/cjs/loader.js:538:12)
17:58:34     at Function.Module._load (internal/modules/cjs/loader.js:530:3)
17:58:34     at Module.require (internal/modules/cjs/loader.js:637:17)
17:58:34     at require (internal/modules/cjs/helpers.js:20:18)
17:58:34 Building addon in /data/iojs/build/workspace/node-test-commit-linuxone/nodes/rhel72-s390x/test/addons/05_wrapping_c_objects
17:58:34 make[2]: *** [out/doc/api/path.html] Error 1

https://ci.nodejs.org/job/node-test-commit-linuxone/3483/nodes=rhel72-s390x/consoleFull

CI-lite: https://ci.nodejs.org/job/node-test-pull-request-lite-pipeline/411/

Node.js Collaborators, please, add 👍 here if you approve fast-tracking.

It seems we have one more similar regression case: HTML entities verbatim rendering:

https://nodejs.org/download/nightly/v11.0.0-nightly20180803d68f946fe4/docs/api/inspector.html#inspector_event_lt_inspector_protocol_method_gt

Not sure if this should be fixed in this PR or in a new one.

@rubys ... just a nit the prefix for the commit message here should be tools: rather than doc:

Also, just as a convention, it is helpful to include a [Squash] or [Fixup] prefix on commits that need to be squashed prior to landing.

My pull request for https://www.npmjs.com/package/mdast-util-to-hast has been accepted. Updating to the latest version of this dependency should fix everything. In fact, I should be able to back out some of the work that is currently committed. I'll run some tests and see what needs to be done from there.

@rubys ... is this ready to go?

@jasnell: this PR is an accumulated set of workarounds to a remark bug, it certainly can go in now, but will all need to be backed out when these pull requests land: #22140 (comment)

CI: https://ci.nodejs.org/job/node-test-pull-request/16411/

My read is that the freebsd failure is unrelated. Unless there are any objections, I'll land this PR tomorrow.

Landed in 59f8276