FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

[12.x backport] module: remove experimental modules warning by guybedford · Pull Request #35749 · nodejs/node · GitHub

/ node Public

[12.x backport] module: remove experimental modules warning - #35749

Closed
guybedford wants to merge 1 commit into
nodejs:v12.x-stagingfrom
guybedford:12-backport-experimental-remove
Closed

[12.x backport] module: remove experimental modules warning#35749
guybedford wants to merge 1 commit into
nodejs:v12.x-stagingfrom
guybedford:12-backport-experimental-remove

Conversation

guybedford commented Oct 22, 2020
edited
Loading

Copy link
Copy Markdown
Contributor

This backports the PR with the removal of the experimental warning from #31974.

This seems important as many users are unaware the 12.x implementation provides the same stable base-level modules functionality.

It would be nice to get the approvals here in the mean time, but this should probably only land once the two previously left out backports from the last 12 release have landed:

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • commit message follows commit guidelines

PR-URL: nodejs#31974
Reviewed-By: Myles Borins <myles.borins@gmail.com>
Reviewed-By: Jan Krems <jan.krems@gmail.com>
Reviewed-By: Geoffrey Booth <webmaster@geoffreybooth.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
nodejs-github-bot added process Issues and PRs related to the process subsystem. v12.x labels Oct 22, 2020

guybedford commented Oct 23, 2020
edited
Loading

Copy link
Copy Markdown
Contributor Author

Superseded by #35757.

guybedford closed this Oct 23, 2020
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

process Issues and PRs related to the process subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL