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.

Sign up for GitHub

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

Jump to bottom

doc: remove section on export control #617

Merged
merged 1 commit into from
Jul 12, 2021
Merged

doc: remove section on export control #617

merged 1 commit into from
Jul 12, 2021

Conversation

mhdawson
Copy link
Member

Signed-off-by: Michael Dawson mdawson@devrus.com

This is long out of date and I've been told that the OpenJS foundation has new/different processes for handling this through the Linux Foundation.

@mhdawson
doc: remove section on export control
e348282 
Signed-off-by: Michael Dawson <mdawson@devrus.com>
@mhdawson
Copy link
Member Author

@jorydotcom, @brianwarner FYI.

@mhdawson
Copy link
Member Author

@nodejs/tsc, @nodejs/community-committee would be good to get some reviews so that this can land.

@mmarchini
Copy link
Contributor

the OpenJS foundation has new/different processes for handling this through the Linux Foundation

Care to elaborate? It might help folks get peace of mind to approve the PR.

@mhdawson
Copy link
Member Author

@mmarchini I don't have the details @brianwarner might, but at this point the doc we have is long out of date so wanted to remove it.

@bnb
Copy link
Contributor

bnb commented Jun 24, 2021

Happy to review pending @mmarchini's request for context

@mhdawson mhdawson mentioned this pull request Jun 29, 2021
Closed
mcollina
mcollina approved these changes Jul 1, 2021
View reviewed changes
Copy link
Member

@mcollina mcollina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

This was referenced Jul 5, 2021
Closed
Closed
@mhdawson
Copy link
Member Author

mhdawson commented Jul 8, 2021

@brianwarner, @jorydotcom people have asked that at least one of you sign off on this before they are comfortable that we remove it.

@brianwarner
Copy link
Contributor

Yep, looks fine, this is replaced by https://openjsf.org/export.

@mhdawson mhdawson mentioned this pull request Jul 12, 2021
Closed
@mhdawson
Copy link
Member Author

k, we have 2 TSC and 2 CommComm approvals now so I'm going to land.

@mhdawson mhdawson merged commit ba98e50 into nodejs:main Jul 12, 2021
aduh95 added a commit to nodejs/email that referenced this pull request Jan 31, 2024
@aduh95
remove crypto-export@iojs.org alias
e6c7ada 
Refs: nodejs/admin#617
@aduh95 aduh95 mentioned this pull request Jan 31, 2024
Merged
richardlau added a commit to nodejs/TSC that referenced this pull request Feb 1, 2024
@richardlau
doc: remove references to crypto-exports list
89fa3d8 
The `crypto-exports` mailing list is no longer used and is being removed.

Refs: nodejs/email#238
Refs: nodejs/admin#617
@richardlau richardlau mentioned this pull request Feb 1, 2024
Merged
richardlau added a commit to nodejs/TSC that referenced this pull request Feb 1, 2024
@richardlau
doc: remove references to crypto-export list
49d28e3 
The `crypto-export` mailing list is no longer used and is being removed.

Refs: nodejs/email#238
Refs: nodejs/admin#617
richardlau pushed a commit to nodejs/email that referenced this pull request Feb 2, 2024
@aduh95
remove crypto-export@iojs.org alias (#238)
4e1dfc3 
Refs: nodejs/admin#617
aduh95 pushed a commit to nodejs/TSC that referenced this pull request Feb 2, 2024
@richardlau
doc: remove references to crypto-export list (#1499)
52fffee 
The `crypto-export` mailing list is no longer used and is being removed.

Refs: nodejs/email#238
Refs: nodejs/admin#617
Johnhvy pushed a commit to Johnhvy/NodeJS-Email that referenced this pull request Jul 16, 2024
@aduh95
remove crypto-export@iojs.org alias (#238)
f33823e 
Refs: nodejs/admin#617
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mcollina mcollina mcollina approved these changes

@joesepi joesepi joesepi approved these changes

@bnb bnb bnb approved these changes

@brianwarner brianwarner brianwarner approved these changes

@BethGriggs BethGriggs BethGriggs approved these changes

Assignees
No one assigned
Labels
cc-agenda tsc-agenda
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
7 participants
@mhdawson @mmarchini @bnb @brianwarner @mcollina @joesepi @BethGriggs