Skip to content

Update Node.js core modules#62201

Merged
jakebailey merged 1 commit intomicrosoft:mainfrom
Renegade334:update-node-modules
Aug 11, 2025
Merged

Update Node.js core modules#62201
jakebailey merged 1 commit intomicrosoft:mainfrom
Renegade334:update-node-modules

Conversation

@Renegade334
Copy link
Contributor

Synchronises some upstream changes to builtin modules.

},
"unresolvedImports": [
"node:zlib",
"node:quic",
Copy link
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
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?

Copy link
Contributor Author

@Renegade334 Renegade334 Aug 5, 2025

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 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 Renegade334 deleted the update-node-modules branch August 11, 2025 21:26
johnfav03 pushed a commit to johnfav03/TypeScript that referenced this pull request Aug 14, 2025
@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.

2 participants