Skip to content

Update Node.js core modules - #62201

Merged
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
Renegade334:update-node-modules
Aug 11, 2025
Merged

Update Node.js core modules#62201
Jake Bailey (jakebailey) merged 1 commit into
microsoft:mainfrom
Renegade334:update-node-modules

Conversation

@Renegade334

Copy link
Copy Markdown
Contributor

Synchronises some upstream changes to builtin modules.

},
"unresolvedImports": [
"node:zlib",
"node:quic",

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I was very confused why everything in this list got changed until I realized it's because one module was removed and one was added, so the diff makes it look like the code shifted

]
}

ts.JsTyping.discoverTypings::

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

This entire log was removed; is the test testing something that was removed and needs to be updated?

@Renegade334 René (Renegade334) Aug 5, 2025

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

It's due to a removed "unqualified" module from the list (test/mock_loader), which was an internal module that was previously populated onto the public list, so there's one less resolution.

@github-project-automation github-project-automation Bot moved this from Not started to Needs merge in PR Backlog Aug 11, 2025
@jakebailey
Jake Bailey (jakebailey) merged commit e39d577 into microsoft:main Aug 11, 2025
33 checks passed
@github-project-automation github-project-automation Bot moved this from Needs merge to Done in PR Backlog Aug 11, 2025
@Renegade334
René (Renegade334) deleted the update-node-modules branch August 11, 2025 21:26
@microsoft Microsoft (microsoft) locked as resolved and limited conversation to collaborators Feb 10, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

3 participants