X Tutup
The Wayback Machine - https://web.archive.org/web/20201025234027/https://github.com/nodejs/node/pull/35781
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

doc: mark ECMAScript modules implementation as stable #35781

Open
wants to merge 5 commits into
base: master
from

Conversation

@guybedford
Copy link
Contributor

@guybedford guybedford commented Oct 23, 2020

Per discussion at the last meeting this is a PR to mark the entire ECMAScript modules implementation as stable while keeping the individual subfeatures unstable as appropriate.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines
@nodejs-github-bot
Copy link

@nodejs-github-bot nodejs-github-bot commented Oct 23, 2020

Review requested:

@guybedford guybedford changed the title doc: mark modules implementation as stable doc: mark ECMAScript modules implementation as stable Oct 23, 2020
Copy link
Member

@MylesBorins MylesBorins left a comment

As mentioned in the meeting I would like to see the commonjs named exports bake a bit longer before landing this.

Thoughts?

doc/api/esm.md Outdated Show resolved Hide resolved
@guybedford
Copy link
Contributor Author

@guybedford guybedford commented Oct 23, 2020

My main reason for posting this now was that it would be nice to include in the 12 backport process, since the two milestones here go together in my mind.

@guybedford
Copy link
Contributor Author

@guybedford guybedford commented Oct 23, 2020

I'm totally fine with delaying this explicit "announcement" though too. It would be good to define what we mean by settled as well.

doc/api/esm.md Outdated Show resolved Hide resolved
guybedford added 3 commits Oct 24, 2020
Copy link
Member

@mcollina mcollina left a comment

lgtm

Copy link
Member

@benjamingr benjamingr left a comment

Good job @nodejs/modules-active-members !!!

I know this was a long and hard journey.

@Trott
Trott approved these changes Oct 24, 2020
@targos
targos approved these changes Oct 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

8 participants
You can’t perform that action at this time.
X Tutup