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 probe description #15397

Closed
wants to merge 1 commit into from

Conversation

evanlucas
Copy link
Contributor

This fixes a typo in a probe description added in
dc1996d.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)

src

This fixes a typo in a probe description added in
dc1996d.
@nodejs-github-bot nodejs-github-bot added the c++ Issues and PRs that require attention from people who are familiar with C++. label Sep 13, 2017
@evanlucas
Copy link
Contributor Author

@jasnell
Copy link
Member

jasnell commented Sep 14, 2017

This shouldn't need to wait the 48 hours

jasnell pushed a commit that referenced this pull request Sep 14, 2017
This fixes a typo in a probe description added in
dc1996d.

PR-URL: #15397
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@jasnell
Copy link
Member

jasnell commented Sep 14, 2017

Landed in d195a06

@jasnell jasnell closed this Sep 14, 2017
@evanlucas evanlucas deleted the ustacktypo branch September 15, 2017 18:23
@evanlucas
Copy link
Contributor Author

Thanks for landing James!

addaleax pushed a commit to addaleax/ayo that referenced this pull request Sep 17, 2017
This fixes a typo in a probe description added in
dc1996d.

PR-URL: nodejs/node#15397
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@jasnell
Copy link
Member

jasnell commented Sep 20, 2017

Ping @MylesBorins .. this needs to land along with the V8 6.1 bump in v8.x

Qard pushed a commit to Qard/ayo that referenced this pull request Sep 21, 2017
This fixes a typo in a probe description added in
dc1996d.

PR-URL: nodejs/node#15397
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
MylesBorins pushed a commit that referenced this pull request Sep 29, 2017
This fixes a typo in a probe description added in
dc1996d.

PR-URL: #15397
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
MylesBorins pushed a commit that referenced this pull request Oct 3, 2017
This fixes a typo in a probe description added in
dc1996d.

PR-URL: #15397
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@MylesBorins MylesBorins mentioned this pull request Oct 3, 2017
MylesBorins pushed a commit that referenced this pull request Oct 3, 2017
This fixes a typo in a probe description added in
dc1996d.

PR-URL: #15397
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c++ Issues and PRs that require attention from people who are familiar with C++.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants