Skip to content

doc: revise strict mode text in assert - #28285

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:revise-strict-mode-yo
Jun 21, 2019
Merged

doc: revise strict mode text in assert#28285
Trott merged 1 commit into
nodejs:masterfrom
Trott:revise-strict-mode-yo

Conversation

@Trott

Copy link
Copy Markdown
Member

Revise the text describing strict mode.

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 assert Issues and PRs related to the assert subsystem. doc Issues and PRs related to the documentations. labels Jun 19, 2019
@Trott
Trottforce-pushed the revise-strict-mode-yo branch from f96e5f4 to cfbd5f5CompareJune 19, 2019 05:16
@Trott

Copy link
Copy Markdown
MemberAuthor

@TrottTrott added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Jun 19, 2019
Revise the text describing `strict` mode.
PR-URL: nodejs#28285
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
@Trott
Trottforce-pushed the revise-strict-mode-yo branch from cfbd5f5 to 3af27f8CompareJune 21, 2019 09:30
@Trott

Copy link
Copy Markdown
MemberAuthor

Landed in 3af27f8

@Trott
Trott merged commit 3af27f8 into nodejs:masterJun 21, 2019
targos pushed a commit that referenced this pull request Jul 2, 2019
Revise the text describing `strict` mode.
PR-URL: #28285
Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
@targostargos mentioned this pull request Jul 2, 2019
@Trott
Trott deleted the revise-strict-mode-yo branch January 13, 2022 22:51
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

assertIssues and PRs related to the assert subsystem.author readyPRs that have at least one approval, no pending requests for changes, and a CI started.docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@Trott@BridgeAR@trivikr@nodejs-github-bot