Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v9.x backport] doc: add new documentation rule #19189

Closed
wants to merge 1 commit into from
Closed

[v9.x backport] doc: add new documentation rule #19189

wants to merge 1 commit into from

Conversation

estrada9166
Copy link
Contributor

Backporting doc changes.
Refs: #18726 (comment)

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

Doc

@nodejs-github-bot nodejs-github-bot added build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project. v9.x labels Mar 6, 2018
@MylesBorins MylesBorins force-pushed the v9.x-staging branch 2 times, most recently from 4e9ac10 to 50d1233 Compare March 7, 2018 17:31
@MylesBorins
Copy link
Contributor

landed in cde6671

@MylesBorins MylesBorins closed this Mar 7, 2018
MylesBorins pushed a commit that referenced this pull request Mar 7, 2018
Add 80 characters limit to docs.
Change docs to fit 80 characters per row.

Backport-PR-URL: #19189
PR-URL: #18726
Fixes: #18703
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
@MylesBorins MylesBorins reopened this Mar 7, 2018
@MylesBorins
Copy link
Contributor

MylesBorins commented Mar 7, 2018

Failing on OSX when running make lint-md with

Running Markdown linter on misc docs...
BUILDING.md
  1:1  error  Error: Cannot parse file `.remarkrc`

edit: backed out of staging. Please rebase and attempt to get it working locally. Thanks!

@estrada9166
Copy link
Contributor Author

estrada9166 commented Mar 7, 2018

@MylesBorins I thought it was solved in this PR and the v9.x-staging already have it, do you know what can be happening?

Backporting doc changes.
PR: #18726
Refs: #18726 (comment)
@estrada9166
Copy link
Contributor Author

@joyeecheung Do you know what can be happening? I thought it was solved in your PR

@joyeecheung
Copy link
Member

@estrada9166 That PR fixes make lint-md-build, not make lint-md - to fix the dependency one will still have to run make lint-md-build manually, because that requires internet access and can't be just run automatically.

@MylesBorins
Copy link
Contributor

ping @estrada9166

@estrada9166
Copy link
Contributor Author

estrada9166 commented Mar 21, 2018

@MylesBorins I'm waiting some comments on this PR, it solve the error that you got

Running Markdown linter on misc docs...
BUILDING.md
  1:1  error  Error: Cannot parse file `.remarkrc`

targos pushed a commit that referenced this pull request Mar 24, 2018
Add 80 characters limit to docs.
Change docs to fit 80 characters per row.

Backport-PR-URL: #19189
PR-URL: #18726
Fixes: #18703
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Matteo Collina <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Anatoli Papirovski <[email protected]>
@targos
Copy link
Member

targos commented Mar 24, 2018

Fixed conflict and landed in 26e97a1

@targos targos closed this Mar 24, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
build Issues and PRs related to build files or the CI. doc Issues and PRs related to the documentations. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants