Skip to content
This repository was archived by the owner on Oct 18, 2023. It is now read-only.

fix: provide file path to dynamic import transform#33

Merged
privatenumber merged 4 commits intoesbuild-kit:nextfrom
mshima:patch-1
Sep 30, 2022
Merged

fix: provide file path to dynamic import transform#33
privatenumber merged 4 commits intoesbuild-kit:nextfrom
mshima:patch-1

Conversation

@mshima
Copy link
Copy Markdown
Contributor

@mshima mshima commented Sep 26, 2022

Related to #32.

Doesn't completely fix the mentioned bug, but looks to be a valid check.

@mshima mshima marked this pull request as draft September 26, 2022 19:29
@mshima mshima marked this pull request as ready for review September 28, 2022 12:08
@privatenumber privatenumber changed the title build dynamic code if dynamic import is found fix: provide file path to dynamic import transform Sep 29, 2022
@privatenumber
Copy link
Copy Markdown
Member

This is a breaking change. Will batch with some other changes.

@privatenumber privatenumber changed the base branch from develop to next September 29, 2022 13:37
@privatenumber privatenumber linked an issue Sep 30, 2022 that may be closed by this pull request
@privatenumber privatenumber merged commit 635f6cc into esbuild-kit:next Sep 30, 2022
@mshima mshima deleted the patch-1 branch September 30, 2022 14:28
privatenumber added a commit that referenced this pull request Sep 30, 2022
BREAKING CHANGE: transformDynamicImport now requires file path

Co-authored-by: Hiroki Osame <hiroki.osame@gmail.com>
privatenumber added a commit that referenced this pull request Oct 2, 2022
BREAKING CHANGE: transformDynamicImport now requires file path

Co-authored-by: Hiroki Osame <hiroki.osame@gmail.com>
privatenumber added a commit that referenced this pull request Oct 4, 2022
BREAKING CHANGE: transformDynamicImport now requires file path

Co-authored-by: Hiroki Osame <hiroki.osame@gmail.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

esm-loader breaks stack trace for esm files

2 participants