Skip to content

doc: simplify and clarify ReferenceError material in errors.md - #34151

Merged
Trott merged 0 commit into
nodejs:masterfrom
Trott:error-considered
Jul 3, 2020
Merged

doc: simplify and clarify ReferenceError material in errors.md#34151
Trott merged 0 commit into
nodejs:masterfrom
Trott:error-considered

Conversation

@Trott

@TrottTrott commented Jul 1, 2020

Copy link
Copy Markdown
Member

"should always be considered" -> "indicate"

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

@nodejs-github-botnodejs-github-bot added doc Issues and PRs related to the documentations. errors Issues and PRs related to JavaScript errors originated in Node.js core. labels Jul 1, 2020
@Trott

Trott commented Jul 2, 2020

Copy link
Copy Markdown
MemberAuthor

@nodejs/documentation

@Trott

Trott commented Jul 3, 2020

Copy link
Copy Markdown
MemberAuthor

@nodejs/collaborators This could use some reviews. Sorry to ping widely but we don't have an errors group to ping...

@Trott

Trott commented Jul 3, 2020

Copy link
Copy Markdown
MemberAuthor

Landed in dfeeb3c

@TrottTrott closed this Jul 3, 2020
@Trott
Trottforce-pushed the error-considered branch from f233e4e to dfeeb3cCompareJuly 3, 2020 14:34
@Trott
Trott merged commit dfeeb3c into nodejs:masterJul 3, 2020
@Trott
Trott deleted the error-considered branch July 3, 2020 14:34
MylesBorins pushed a commit that referenced this pull request Jul 14, 2020
"should always be considered" -> "indicate"
PR-URL: #34151
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Andrey Pechkurov <apechkurov@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Jul 14, 2020
MylesBorins pushed a commit that referenced this pull request Jul 16, 2020
"should always be considered" -> "indicate"
PR-URL: #34151
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Andrey Pechkurov <apechkurov@gmail.com>
addaleax pushed a commit that referenced this pull request Sep 22, 2020
"should always be considered" -> "indicate"
PR-URL: #34151
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Anto Aravinth <anto.aravinth.cse@gmail.com>
Reviewed-By: Christian Clauss <cclauss@me.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Chengzhong Wu <legendecas@gmail.com>
Reviewed-By: Andrey Pechkurov <apechkurov@gmail.com>
@codebyterecodebytere mentioned this pull request Sep 28, 2020
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.errorsIssues and PRs related to JavaScript errors originated in Node.js core.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@Trott@antsmartian@lpinca@cclauss@legendecas@Flarna@puzpuzpuz@nodejs-github-bot