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

doc: remove obsolete cc recommendations #39181

Merged
merged 1 commit into from Jun 30, 2021
Merged

doc: remove obsolete cc recommendations #39181

merged 1 commit into from Jun 30, 2021

Conversation

@Trott
Copy link
Member

@Trott Trott commented Jun 28, 2021

In 2016, Ali added the instruction to cc him on particular V8 bugs and
the suggestion to contact him for certain problems. However, I'm pretty
sure Ali has not been working on V8 for 2+ years at this point. I'm
guessing those instructions are obsolete at this point. (I'll ping him
in the PR for this change to confirm, though.)

Also: Remove a parenthetical and add a comma, because I can't imagine
opening a separate pull request to add a comma. (OK, I'm lying, I
totally can and if anyone has done that in the past, it's probably me.)

@github-actions github-actions bot added the doc label Jun 28, 2021
@Trott
Copy link
Member Author

@Trott Trott commented Jun 28, 2021

@ofrobots Am I guessing correctly that you're not the person to CC/ping on these things anymore? Or should I leave this text alone?

Copy link
Member

@mhdawson mhdawson left a comment

LGTM

@lpinca
lpinca approved these changes Jun 29, 2021
In 2016, Ali added the instruction to cc him on particular V8 bugs and
the suggestion to contact him for certain problems. However, I'm pretty
sure Ali has not been working on V8 for 2+ years at this point. I'm
guessing those instructions are obsolete at this point. (I'll ping him
in the PR for this change to confirm, though.)

Also: Remove a parenthetical and add a comma, because I can't imagine
opening a separate pull request to add a comma. (OK, I'm lying, I
totally can and if anyone has done that in the past, it's probably me.)

PR-URL: #39181
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@Trott Trott force-pushed the Trott:ofrobots branch from 58681e0 to 593212c Jun 30, 2021
@Trott
Copy link
Member Author

@Trott Trott commented Jun 30, 2021

Landed in 593212c

@Trott Trott merged commit 593212c into nodejs:master Jun 30, 2021
13 checks passed
13 checks passed
@github-actions
build-tarball
Details
@github-actions
lint-commit-message
Details
@github-actions
build-windows
Details
@github-actions
lint-addon-docs
Details
@github-actions
build-docs
Details
@github-actions
test-linux
Details
@github-actions
lint-cpp
Details
@github-actions
lint-md
Details
@github-actions
lint-js
Details
@github-actions
lint-py
Details
@github-actions
lint-sh
Details
@github-actions
lint-codeowners
Details
@github-actions
lint-pr-url
Details
@Trott Trott deleted the Trott:ofrobots branch Jun 30, 2021
targos added a commit that referenced this pull request Jul 11, 2021
In 2016, Ali added the instruction to cc him on particular V8 bugs and
the suggestion to contact him for certain problems. However, I'm pretty
sure Ali has not been working on V8 for 2+ years at this point. I'm
guessing those instructions are obsolete at this point. (I'll ping him
in the PR for this change to confirm, though.)

Also: Remove a parenthetical and add a comma, because I can't imagine
opening a separate pull request to add a comma. (OK, I'm lying, I
totally can and if anyone has done that in the past, it's probably me.)

PR-URL: #39181
Reviewed-By: Michael Dawson <midawson@redhat.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
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
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

4 participants