FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: format ECMA-262 with a hyphen by cjihrig · Pull Request #28309 · nodejs/node · GitHub

/ node Public

doc: format ECMA-262 with a hyphen - #28309

Merged
cjihrig merged 1 commit into
nodejs:masterfrom
cjihrig:ecma
Jun 22, 2019
Merged

doc: format ECMA-262 with a hyphen#28309
cjihrig merged 1 commit into
nodejs:masterfrom
cjihrig:ecma

Conversation

cjihrig commented Jun 20, 2019

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

nodejs-github-bot added doc Issues and PRs related to the documentations. node-api Issues and PRs related to the Node-API. labels Jun 20, 2019
ZYSzys added the author ready PRs that have at least one approval, no outstanding review comments, and a CI started. label Jun 20, 2019

ZYSzys commented Jun 20, 2019
edited by cjihrig
Loading

Copy link
Copy Markdown
Member

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

EDIT(cjihrig): CI was green.

Copy link
Copy Markdown
Member

Is it worth adding this to remark-preset-lint-node as a prohibited string (like we do for RFC <number>)?

cjihrig commented Jun 20, 2019

Copy link
Copy Markdown
Contributor Author

Is it worth adding this to remark-preset-lint-node as a prohibited string (like we do for RFC <number>)?

Yes! I tried to do that, but couldn't remember where those settings lived.

PR-URL: nodejs#28309
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
cjihrig merged commit f00d5bc into nodejs:master Jun 22, 2019
cjihrig deleted the ecma branch June 22, 2019 16:01
targos pushed a commit that referenced this pull request Jul 2, 2019
PR-URL: #28309
Reviewed-By: Gus Caplan <me@gus.host>
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Richard Lau <riclau@uk.ibm.com>
Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com>
Reviewed-By: Tiancheng "Timothy" Gu <timothygu99@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
targos mentioned this pull request Jul 2, 2019
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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 outstanding review comments, and a CI started. doc Issues and PRs related to the documentations. node-api Issues and PRs related to the Node-API.

Projects

None yet

Development

Successfully merging this pull request may close these issues.


Back | FazBrowse Home | New Git URL