{"action":"created","issue":{"url":"https://api.github.com/repos/nodejs/node/issues/8551","repository_url":"https://api.github.com/repos/nodejs/node","labels_url":"https://api.github.com/repos/nodejs/node/issues/8551/labels{/name}","comments_url":"https://api.github.com/repos/nodejs/node/issues/8551/comments","events_url":"https://api.github.com/repos/nodejs/node/issues/8551/events","html_url":"https://github.com/nodejs/node/pull/8551","id":177265396,"number":8551,"title":"Introduce 'make mdlint' command and tooling, some doc fixes, update remark-lint rules","user":{"login":"ChALkeR","id":291301,"avatar_url":"https://avatars.githubusercontent.com/u/291301?v=3","gravatar_id":"","url":"https://api.github.com/users/ChALkeR","html_url":"https://github.com/ChALkeR","followers_url":"https://api.github.com/users/ChALkeR/followers","following_url":"https://api.github.com/users/ChALkeR/following{/other_user}","gists_url":"https://api.github.com/users/ChALkeR/gists{/gist_id}","starred_url":"https://api.github.com/users/ChALkeR/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ChALkeR/subscriptions","organizations_url":"https://api.github.com/users/ChALkeR/orgs","repos_url":"https://api.github.com/users/ChALkeR/repos","events_url":"https://api.github.com/users/ChALkeR/events{/privacy}","received_events_url":"https://api.github.com/users/ChALkeR/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/nodejs/node/labels/build","name":"build","color":"eb6420"},{"url":"https://api.github.com/repos/nodejs/node/labels/doc","name":"doc","color":"006b75"},{"url":"https://api.github.com/repos/nodejs/node/labels/dont-land-on-v4.x","name":"dont-land-on-v4.x","color":"fad8c7"},{"url":"https://api.github.com/repos/nodejs/node/labels/dont-land-on-v6.x","name":"dont-land-on-v6.x","color":"bfdadc"},{"url":"https://api.github.com/repos/nodejs/node/labels/dont-land-on-v7.x","name":"dont-land-on-v7.x","color":"fbca04"},{"url":"https://api.github.com/repos/nodejs/node/labels/tools","name":"tools","color":"d4c5f9"}],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2016-09-15T19:22:54Z","updated_at":"2016-09-20T11:55:08Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/nodejs/node/pulls/8551","html_url":"https://github.com/nodejs/node/pull/8551","diff_url":"https://github.com/nodejs/node/pull/8551.diff","patch_url":"https://github.com/nodejs/node/pull/8551.patch"},"body":"##### Checklist\r\n<!-- Remove items that do not apply. For completed items, change [ ] to [x]. -->\r\n\r\n- [x] `make -j4 test` (UNIX), or `vcbuild test nosign` (Windows) passes\r\n- [x] documentation is changed or added\r\n- [x] commit message follows commit guidelines\r\n\r\n##### Affected core subsystem(s)\r\ndoc, tools\r\n\r\n##### Description of change\r\n\r\nWhat this does:\r\n * clean up `icu/README.md` formatting — extracted to #8660,\r\n * ~~fix an error in [doc/api/buffer.md](https://github.com/nodejs/node/blob/master/doc/api/buffer.md#buffers-and-typedarray), (see the `TypedArray.from()` link)~~ — extracted to #8553,\r\n * update the `.remarkrc` file to work with new `remark-cli` and `remark-lint`,\r\n * add `remark-cli` and `remark-lint` to `tools/remark-cli`, introducing new `make mdlint` command.\r\n\r\n`make mdlint` now lints all docs, `make test` now executes `make mdlint`.\r\n\r\nA complete `make mdlint` check takes 8-9 seconds on my notebook.\r\n\r\nPrevious work in #7729.\r\n\r\n/cc @Trott, @jasnell, @addaleax.\r\n\r\nUpd: I extracted the `buffer.md` fix to #8553, as that one fixes a documentation error and could land faster."},"comment":{"url":"https://api.github.com/repos/nodejs/node/issues/comments/248279970","html_url":"https://github.com/nodejs/node/pull/8551#issuecomment-248279970","issue_url":"https://api.github.com/repos/nodejs/node/issues/8551","id":248279970,"user":{"login":"addaleax","id":899444,"avatar_url":"https://avatars.githubusercontent.com/u/899444?v=3","gravatar_id":"","url":"https://api.github.com/users/addaleax","html_url":"https://github.com/addaleax","followers_url":"https://api.github.com/users/addaleax/followers","following_url":"https://api.github.com/users/addaleax/following{/other_user}","gists_url":"https://api.github.com/users/addaleax/gists{/gist_id}","starred_url":"https://api.github.com/users/addaleax/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/addaleax/subscriptions","organizations_url":"https://api.github.com/users/addaleax/orgs","repos_url":"https://api.github.com/users/addaleax/repos","events_url":"https://api.github.com/users/addaleax/events{/privacy}","received_events_url":"https://api.github.com/users/addaleax/received_events","type":"User","site_admin":false},"created_at":"2016-09-20T11:55:08Z","updated_at":"2016-09-20T11:55:08Z","body":"This should probably get all the `dont-land-on-v?.x` labels because this is an almost purely additional change and we’re not eager to backport the doc cleanups, right? I’m setting them but feel free to remove if you think that’s not a good idea.\r\n\r\nAlso, did you check whether the dependencies here end up in the distribution tarballs?"}}
{"actor":{"display_login":"addaleax"}}