Skip to content
This repository has been archived by the owner on Jul 31, 2018. It is now read-only.

meta: note that the repo and process has been archived #64

Closed
wants to merge 1 commit into from

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Sep 13, 2017

See nodejs/TSC#335

Ping @nodejs/tsc

Copy link
Member

@Trott Trott left a comment

Choose a reason for hiding this comment

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

LGTM. Not sure people are going to understand that "decided to archive the node-eps repository and process" means "don't use this". (The word I have questions about is "archive". Might be better as "discontinue" for clarity?) But I'm nit-picking. We can always tweak the wording if it actually ever does cause confusion.

@unional
Copy link

unional commented Sep 13, 2017

What is the preferred mechanism for discussing proposals such as #60 moving forward?

It would be good to add some actionable steps in the message. 🌷

@jasnell
Copy link
Member Author

jasnell commented Sep 13, 2017

Pull requests against master or working in other forks the way we've done with things like http2, Node-ChakraCore, etc

@unional
Copy link

unional commented Sep 13, 2017

Pull requests against master or working in other forks the way we've done with things like http2, Node-ChakraCore, etc

On https://github.com/nodejs/node directly?

@jasnell
Copy link
Member Author

jasnell commented Sep 13, 2017

Yes. Or in working repos such as nodejs/http2

@@ -1,5 +1,9 @@
# Node.js Enhancement Proposals

**NOTE**: The Node.js TSC has [recently](https://github.com/nodejs/TSC/issues/335)
decided to archive the node-eps repository and process. See https://github.com/nodejs/TSC/issues/335

Choose a reason for hiding this comment

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

Nit: Linking the same issue twice in two consecutive lines kinda feels redundant.

@thefourtheye
Copy link

Perhaps we should list all the repos under our roof and explain what they are for, in our main README.md.

@bengl
Copy link
Member

bengl commented Sep 23, 2017

Can (Archived/Inactive) be added to this project's description? That would be helpful in the several contexts in which the description is shown.

@Trott
Copy link
Member

Trott commented Jun 13, 2018

Since GitHub offers genuine read-only archiving now, I think this is obsolete. Feel free to re-open or comment if I'm wrong.

@Trott Trott closed this Jun 13, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants