Skip to content

Comments

fix: moduleResolution: bundler support in tsconfig.json#50289

Closed
kvnwolf wants to merge 3 commits intovercel:canaryfrom
kvnwolf:canary
Closed

fix: moduleResolution: bundler support in tsconfig.json#50289
kvnwolf wants to merge 3 commits intovercel:canaryfrom
kvnwolf:canary

Conversation

@kvnwolf
Copy link
Contributor

@kvnwolf kvnwolf commented May 24, 2023

Fixes #47858
link NEXT-1281

Copy link
Member

@styfle styfle left a comment

Choose a reason for hiding this comment

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

Thanks!

Can you add a test to confirm this work?

See #36189 for an example

@kvnwolf
Copy link
Contributor Author

kvnwolf commented May 26, 2023

Thanks!

Can you add a test to confirm this work?

See #36189 for an example

Tests added here

@styfle
Copy link
Member

styfle commented May 26, 2023

Looks like CI is failing. Can we make sure this change is compatible with TS 4 as well?

@styfle styfle changed the title Add moduleResolution: bundler support fix: moduleResolution: bundler support in tsconfig.json May 30, 2023
kodiakhq bot pushed a commit that referenced this pull request Jun 8, 2023
We need to upgrade TypeScript to land the following fixes:

- #50289
- #48018

And it should fix async components:
- https://twitter.com/sebsilbermann/status/1664356039876124672

Also see this related TS 5.0 issue:

- microsoft/TypeScript#53402
@kodiakhq kodiakhq bot closed this in #51065 Jun 9, 2023
kodiakhq bot pushed a commit that referenced this pull request Jun 9, 2023
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jul 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[NEXT-1281] Support moduleResolution bundler in tsconfig.json

3 participants