search

nodejs / node-eps

Node.js Enhancement Proposals for discussion on future API additions/changes to Node core
443 stars 66 forks source link

doc: Update README.md to include WIP status #22

Closed mhdawson closed 6 years ago

mhdawson commented 8 years ago

Based on the EP for the stable module ABI I think an additional status would be useful to call out those which we believe are ready for final review versus those that we expect to iterate on before that final review should take place.

jasnell commented 8 years ago

I've always disliked "incubating" as a status. Perhaps just "pre-draft"?

mhdawson commented 8 years ago

or maybe "WIP" for work in progress ? otherwise "pre-draft" is fine with me.

evanlucas commented 8 years ago

I like WIP

jasnell commented 8 years ago

Work in progress works for me too On May 4, 2016 7:10 AM, "Michael Dawson" notifications@github.com wrote:

or maybe "WIP" for workin progress ? otherwise "pre-draft" is fine with me.

— You are receiving this because you commented. Reply to this email directly or view it on GitHub https://github.com/nodejs/node-eps/pull/22#issuecomment-216877144

mhdawson commented 8 years ago

Updated to use WIP

mhdawson commented 8 years ago

Any additional comments ?

@trevnorris I know you had some on the other PR where the change had been accidentally included.

trevnorris commented 8 years ago

I'm not sure if I completely agree with how this works. Documents can land as DRAFT while they're being revised. A WIP to me means that the PR is asking for help to review/fix/etc. That's easily enough placed in the title of the PR. I don't like the idea of a document landing with WIP, since it denotes that it's definitely not something even potentially accepted. A document should land unless it's been agreed upon that it could be implemented.

mhdawson commented 6 years ago

Long since stale. node-eps has been sunset.