Skip to content

test: replace string concatenation to template literals - #14292

Closed
vercentdeng wants to merge 1 commit into
nodejs:masterfrom
vercentdeng:confTest
Closed

test: replace string concatenation to template literals#14292
vercentdeng wants to merge 1 commit into
nodejs:masterfrom
vercentdeng:confTest

Conversation

@vercentdeng

@vercentdeng vercentdeng commented Jul 16, 2017

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines
Affected core subsystem(s)

test: replace string concatenation to template literals

@nodejs-github-bot nodejs-github-bot added async_hooks Issues and PRs related to the async hooks subsystem. test Issues and PRs related to Node.js core tests and test infrastructure. labels Jul 16, 2017
@Trott Trott added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Jul 16, 2017
Trott
Trott approved these changes Jul 16, 2017
View reviewed changes

@Trott Trott left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM if CI is green

benjamingr
benjamingr approved these changes Jul 16, 2017
View reviewed changes

@benjamingr benjamingr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for your contribution and welcome to the project :)

@tniessen tniessen self-assigned this Jul 16, 2017
@vsemozhetbyt

vsemozhetbyt commented Jul 16, 2017

Copy link
Copy Markdown
Contributor

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

@Trott

Trott commented Jul 17, 2017

Copy link
Copy Markdown
Member

CI is green! 🎉

Trott pushed a commit to Trott/io.js that referenced this pull request Jul 17, 2017
@vercentdeng @Trott
test: use template literals in test-writewrap
25f27b7 
Use template literals instead of string concatenation in
test/async-hooks/test-writewrap.js.

PR-URL: nodejs#14292
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
@Trott

Trott commented Jul 17, 2017

Copy link
Copy Markdown
Member

Small change, green CI, 8 approvals, fast-tracking.

Landed in 25f27b7

@Trott Trott closed this Jul 17, 2017
@Trott

Trott commented Jul 17, 2017

Copy link
Copy Markdown
Member

Thanks for the contribution! 🎉

@addaleax addaleax mentioned this pull request Jul 18, 2017
Merged
addaleax pushed a commit that referenced this pull request Jul 18, 2017
@vercentdeng @addaleax
test: use template literals in test-writewrap
9257e7e 
Use template literals instead of string concatenation in
test/async-hooks/test-writewrap.js.

PR-URL: #14292
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Fishrock123 pushed a commit that referenced this pull request Jul 19, 2017
@vercentdeng @Fishrock123
test: use template literals in test-writewrap
bf55439 
Use template literals instead of string concatenation in
test/async-hooks/test-writewrap.js.

PR-URL: #14292
Reviewed-By: Rich Trott <rtrott@gmail.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Trott Trott Trott approved these changes
@benjamingr benjamingr benjamingr approved these changes
@cjihrig cjihrig cjihrig approved these changes
@tniessen tniessen tniessen approved these changes
@gireeshpunathil gireeshpunathil gireeshpunathil approved these changes
@BridgeAR BridgeAR BridgeAR approved these changes
+2 more reviewers
@targos targos targos approved these changes
@gibfahn gibfahn gibfahn approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

@tniessen tniessen

Labels

async_hooks Issues and PRs related to the async hooks subsystem. code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.