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

email: add alias for crypto reporting #104

Closed
wants to merge 2 commits into from
Closed

Conversation

mhdawson
Copy link
Member

No description provided.

Trott

This comment was marked as off-topic.

@Trott
Copy link
Member

Trott commented Jun 21, 2018

Myles appears in the list twice.

Would it be more maintainable if we replaced the listed-out TSC folks with tsc@iojs.org?

What is the purpose of this email alias? Is it mentioned in the recent export doc or something? I thought we determined that all that stuff was not much of a TSC concern as much as a Foundation/Board concern.

@mhdawson
Copy link
Member Author

It is mentioned in the doc: https://github.com/nodejs/admin/blob/master/us-export-control.md which defines who will get emails.

Myles is there because I copied the TSC list and then added the 2 other positions that should be copied. It might be easier to keep it that way than having to update when the TSC list updates (ie remember to remove him each time).

I would have used tsc@iojs.org but I believe we figured out that recursion does not work in the lists and that for now we had to leave them expanded.

@mhdawson
Copy link
Member Author

@Trott see #59 in terms of the limitation on expansion.

@mrhinkle
Copy link

crypto-exports@nodejs.org now forwards to crypto-export@iojs.org as well.

@mhdawson
Copy link
Member Author

@Trott can you take another look, would like to get alias in place since node.js version is already forwarding to it.

Trott

This comment was marked as off-topic.

mhdawson added a commit that referenced this pull request Jun 25, 2018
PR-URL: #104
Reviewed-By: Rich Trott <rtrott@gmail.com>
@mhdawson
Copy link
Member Author

Landed as 7ea0495

@mhdawson mhdawson closed this Jun 25, 2018
Johnhvy added a commit to Johnhvy/NodeJS-Email that referenced this pull request Jul 16, 2024
PR-URL: nodejs/email#104
Reviewed-By: Rich Trott <rtrott@gmail.com>
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.

3 participants