Skip to content

doc: add missing ESM examples to DEP0202 and DEP0207 - #65047

Open
ChocoChip0519 wants to merge 1 commit into
nodejs:mainfrom
ChocoChip0519:fix-deprecations-esm-examples
Open

doc: add missing ESM examples to DEP0202 and DEP0207#65047
ChocoChip0519 wants to merge 1 commit into
nodejs:mainfrom
ChocoChip0519:fix-deprecations-esm-examples

Conversation

@ChocoChip0519

Copy link
Copy Markdown

DEP0202 and DEP0207 each have two consecutive cjs code blocks with no corresponding mjs block, breaking the cjs/mjs pairing convention used elsewhere in this file.

DEP0202 and DEP0207 each have two consecutive cjs code blocks with no
corresponding mjs block, breaking the cjs/mjs pairing convention used
elsewhere in this file.

Signed-off-by: Taeuk Ha <tomcat0519@naver.com>
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/userland-migrations

@nodejs-github-bot nodejs-github-bot added deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations. labels Aug 5, 2026
@avivkeller avivkeller added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 5, 2026
Comment thread doc/api/deprecations.md
Comment on lines +4698 to +4706
```mjs
// Deprecated
server.on('stream', (stream) => {
stream.on('aborted', () => {
// Stream was closed while the writable was still open.
});
});
```

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What value does this bring, your just duplicating the same code

@aduh95 aduh95 removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Aug 11, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

deprecations Issues and PRs related to deprecations. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants