Skip to content

Chore: use template literal in error message #15846

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.

Sign up for GitHub

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

Jump to bottom
Closed
wants to merge 1 commit into from
Closed

Chore: use template literal in error message #15846

wants to merge 1 commit into from

Conversation

@tchon
Copy link

@tchon tchon commented Oct 6, 2017 •
edited
Loading

Description
Checklist
Affected core subsystem(s)
  • tools
@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory. labels Oct 6, 2017
@Trott Trott added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Oct 7, 2017
@gireeshpunathil
Copy link
Member

gireeshpunathil commented Oct 10, 2017

don't know whether this warrants a CI run or not. thoughts?
@tchon
Copy link
Author

tchon commented Oct 10, 2017

@gireeshpunathil I'm alright with running it through CI but I'm unsure how to manually kick it off.
@joyeecheung
Copy link
Member

joyeecheung commented Oct 10, 2017

CI: https://ci.nodejs.org/job/node-test-pull-request/10587/
@gireeshpunathil
Copy link
Member

gireeshpunathil commented Oct 10, 2017

just to clarify - the changed file is a semi-doc file, so I was unsure whether to run the whole test suite to validate its correctness, and hence asked. Now that @joyeecheung has kick-started, so be it.

@tchon - only node.js project members can kick-start CI jobs.
@tchon
Copy link
Author

tchon commented Oct 10, 2017

👍 Thanks folks. I was thinking on seeing if CI could run just for /tools/ but this all works. :-)
@mhdawson
Copy link
Member

mhdawson commented Oct 11, 2017

@gireeshpunathil I believe we pretty much the CI on all changes, I think we have some validation of docs...
@gireeshpunathil
Copy link
Member

gireeshpunathil commented Oct 11, 2017

@mhdawson - understood, thanks.
jasnell pushed a commit that referenced this pull request Oct 13, 2017
@jasnell
tools: use template literal in error message
a26c1bf 
PR-URL: #15846 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@jasnell
Copy link
Member

jasnell commented Oct 13, 2017

Landed in a26c1bf
@jasnell jasnell closed this Oct 13, 2017
addaleax pushed a commit to ayojs/ayo that referenced this pull request Oct 15, 2017
@addaleax
tools: use template literal in error message
72c80cd 
PR-URL: nodejs/node#15846 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
targos pushed a commit that referenced this pull request Oct 18, 2017
@targos
tools: use template literal in error message
917ccc9 
PR-URL: #15846 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 16, 2017
@MylesBorins
tools: use template literal in error message
497170d 
PR-URL: #15846 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@MylesBorins MylesBorins mentioned this pull request Nov 21, 2017
Merged
MylesBorins pushed a commit that referenced this pull request Nov 21, 2017
@MylesBorins
tools: use template literal in error message
dbcca06 
PR-URL: #15846 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
MylesBorins pushed a commit that referenced this pull request Nov 28, 2017
@MylesBorins
tools: use template literal in error message
5b5b5c0 
PR-URL: #15846 Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com> Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. doc Issues and PRs related to the documentations. tools Issues and PRs related to the tools directory.