★ wanayoo — archive 1999 https://github.com/nodejs/node/pull/18280Nouvelle recherche | Portail wanayoo
Skip to content
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.

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

src: fix typo in trace_events_async_hooks.js #18280

Closed
wants to merge 1 commit into from
Closed

src: fix typo in trace_events_async_hooks.js #18280

wants to merge 1 commit into from

Conversation

Copy link
Contributor

@gillesdemey gillesdemey commented Jan 21, 2018

chrome://traceing does not exist

Checklist
Affected core subsystem(s)

none

chrome://traceing does not exist
@gillesdemey gillesdemey changed the title src: fix typo in node_trace_events.cc src: fix typo in trace_events_async_hooks.js Jan 21, 2018
@vsemozhetbyt vsemozhetbyt added async_hooks trace_events fast-track labels Jan 21, 2018
@vsemozhetbyt
Copy link
Contributor

@vsemozhetbyt vsemozhetbyt commented Jan 21, 2018

CI-lite: https://ci.nodejs.org/job/node-test-pull-request-lite/105/ (it seems this suffices as the PR only changes a comment).

vsemozhetbyt pushed a commit that referenced this issue Jan 21, 2018
chrome://traceing does not exist

PR-URL: #18280
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
@vsemozhetbyt
Copy link
Contributor

@vsemozhetbyt vsemozhetbyt commented Jan 21, 2018

Landed in 9545c48

Thank you, @gillesdemey!

@gillesdemey gillesdemey deleted the patch-2 branch Jan 21, 2018
evanlucas pushed a commit that referenced this issue Jan 30, 2018
chrome://traceing does not exist

PR-URL: #18280
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
evanlucas pushed a commit that referenced this issue Jan 30, 2018
chrome://traceing does not exist

PR-URL: #18280
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
@MylesBorins
Copy link
Member

@MylesBorins MylesBorins commented Feb 27, 2018

Should this be backported to v8.x-staging? If yes please follow the guide and raise a backport PR, if not let me know or add the dont-land-on label.

nvm, was able to do it myself.. was on autopilot, sorry for noise

MylesBorins pushed a commit that referenced this issue Feb 27, 2018
chrome://traceing does not exist

PR-URL: #18280
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
MayaLekova pushed a commit to MayaLekova/node that referenced this issue May 8, 2018
chrome://traceing does not exist

PR-URL: nodejs#18280
Reviewed-By: Vse Mozhet Byt <vsemozhetbyt@gmail.com>
Reviewed-By: Jon Moss <me@jonathanmoss.me>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
async_hooks fast-track trace_events
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants