Skip to content

Conversation

cjihrig
Copy link
Contributor

@cjihrig cjihrig commented Jul 6, 2019

Update the API docs to always spell as Unix for consistency.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Jul 6, 2019
Copy link
Member

@bnoordhuis bnoordhuis left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. Would be nice to have a doc lint rule.

@cjihrig
Copy link
Contributor Author

cjihrig commented Jul 6, 2019

Would be nice to have a doc lint rule.

Agreed. It's a bit annoying that the doc rules exist outside of core.

@Trott
Copy link
Member

Trott commented Jul 6, 2019

Would be nice to have a doc lint rule.

Agreed. It's a bit annoying that the doc rules exist outside of core.

Yeah, I'd be for reversing the flow to make this easier on us. In other words, instead of adding a rule to the remark-lint-prohibited-strings and then bringing it into core, it would be great to add the rule to core and then publish remark-lint-prohibited-strings in case other projects (in the nodejs org in particular) want to use our markdown linting rules (which none of them do as far as I know).

That might also involve undoing some or all of the rollup stuff for lint-md too, though.

@cjihrig
Copy link
Contributor Author

cjihrig commented Jul 7, 2019

@cjihrig cjihrig added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jul 7, 2019
Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Update the API docs to always spell as Unix.

PR-URL: nodejs#28576
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@cjihrig cjihrig merged commit e04ee9b into nodejs:master Jul 8, 2019
@cjihrig cjihrig deleted the unix branch July 8, 2019 17:36
targos pushed a commit that referenced this pull request Jul 20, 2019
Update the API docs to always spell as Unix.

PR-URL: #28576
Reviewed-By: Ben Noordhuis <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Reviewed-By: Gireesh Punathil <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
This was referenced Jul 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants