Skip to content

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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants