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

Revert "doc: fix typo in COLLABORATOR_GUIDE.md" #20829

Closed
wants to merge 1 commit into from

Conversation

addaleax
Copy link
Member

This reverts commit 59f71ea.

Quick fix for the following linter error:

COLLABORATOR_GUIDE.md
  846:46-846:66  warning  Use "V8" instead of "v8"  prohibited-strings  remark-lint

⚠ 1 warning

(Despite being a warning, this breaks make test,
and I don’t know how else to address it.)

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

This reverts commit 59f71ea.

Quick fix for the following linter error:

```
COLLABORATOR_GUIDE.md
  846:46-846:66  warning  Use "V8" instead of "v8"  prohibited-strings  remark-lint

⚠ 1 warning
```

(Despite being a warning, this breaks `make test`,
and I don’t know how else to address it.)
@addaleax addaleax requested a review from a team as a code owner May 18, 2018 23:12
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label May 18, 2018
@addaleax addaleax added the fast-track PRs that do not need to wait for 48 hours to land. label May 18, 2018
@addaleax
Copy link
Member Author

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

Please 👍 this comment to approve fast-tracking.

/cc @vsemozhetbyt @BridgeAR

@addaleax
Copy link
Member Author

The single LinuxOne test failure is definitely unrelated to this one-liner, and I’ve seen it in other CI runs today so far too.

@addaleax
Copy link
Member Author

Okay, figured out thanks to @BridgeAR how to address this: make lint-md-clean && make lint-md-build && make lint did the trick. Cheers!

@addaleax addaleax closed this May 18, 2018
@addaleax addaleax deleted the revert-typo branch May 18, 2018 23:44
@BridgeAR BridgeAR mentioned this pull request May 20, 2018
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants