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

gillesdemey
Copy link
Contributor

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 Issues and PRs related to the async hooks subsystem. trace_events Issues and PRs related to V8, Node.js core, and userspace code trace events. fast-track PRs that do not need to wait for 48 hours to land. labels Jan 21, 2018
@vsemozhetbyt
Copy link
Contributor

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 pull request 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

Landed in 9545c48

Thank you, @gillesdemey!

@gillesdemey gillesdemey deleted the patch-2 branch January 21, 2018 15:49
evanlucas pushed a commit that referenced this pull request 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 pull request 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
Contributor

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 pull request 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 pull request 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 Issues and PRs related to the async hooks subsystem. fast-track PRs that do not need to wait for 48 hours to land. trace_events Issues and PRs related to V8, Node.js core, and userspace code trace events.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants