Skip to content

add gibfahn to tsc email #84

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

Closed
wants to merge 2 commits into from
Closed

Conversation

Trott
Copy link
Member

@Trott Trott commented Jan 31, 2018

@gibfahn I notice that you have your gmail address on the mailing list except for your email address for ci-alert which uses your IBM email. Do you want to update that one to gmail (or update the others to IBM)? Or is the use of two email addresses intentional?

jasnell

This comment was marked as off-topic.

targos

This comment was marked as off-topic.

@gibfahn
Copy link
Member

gibfahn commented Feb 1, 2018

Or is the use of two email addresses intentional?

I try to use gmail everywhere, so updating that one makes sense. I'm on holiday atm but will raise a PR when I get round to it.

gibfahn

This comment was marked as off-topic.

@Trott
Copy link
Member Author

Trott commented Feb 1, 2018

I try to use gmail everywhere, so updating that one makes sense. I'm on holiday atm but will raise a PR when I get round to it.

No need. I just made the change in this PR. :-D

ChALkeR

This comment was marked as off-topic.

@jasnell
Copy link
Member

jasnell commented Feb 1, 2018

heh... ask any IBMer anywhere if they'd rather use their IBM email address or literally anything else and the answer will most likely be consistent.

@gibfahn
Copy link
Member

gibfahn commented Feb 1, 2018

heh... ask any IBMer anywhere if they'd rather use their IBM email address or literally anything else and the answer will most likely be consistent.

It's a lot better since Verse, but I've had Gmail for ~10 years, and I have 0 desire to manage more inboxes than I have to...

No need. I just made the change in this PR. :-D

Unacceptable scope creep? 😁

gibfahn

This comment was marked as off-topic.

mhdawson

This comment was marked as off-topic.

mhdawson pushed a commit that referenced this pull request Feb 1, 2018
PR-URL: #84
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
@mhdawson
Copy link
Member

mhdawson commented Feb 1, 2018

Landed as b0dd733

and pushed to email server.

@mhdawson mhdawson closed this Feb 1, 2018
uneoka0114 added a commit to uneoka0114/NodeJS-Email that referenced this pull request Jul 16, 2024
PR-URL: nodejs/email#84
Reviewed-By: James M Snell <[email protected]>
Reviewed-By: Michaël Zasso <[email protected]>
Reviewed-By: Gibson Fahnestock <[email protected]>
Reviewed-By: Сковорода Никита Андреевич <[email protected]>
Reviewed-By: Michael Dawson <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants