Skip to content

fix(plugin/json): tree shake json named exports#3346

Merged
IWANABETHATGUY merged 1 commit intorolldown:mainfrom
hi-ogawa:fix-json-parse-pure
Jan 13, 2025
Merged

fix(plugin/json): tree shake json named exports#3346
IWANABETHATGUY merged 1 commit intorolldown:mainfrom
hi-ogawa:fix-json-parse-pure

Conversation

@hi-ogawa
Copy link
Collaborator

Description

This was once fixed in #2029, but #3103 added another JSON.parse without pure comment.

Someone noticed the same bug on Vite vitejs/vite#19189, so I ported to here.

@hi-ogawa hi-ogawa marked this pull request as ready for review January 13, 2025 05:51
@hi-ogawa hi-ogawa changed the title fix: tree shake json named exports fix(plugin/json): tree shake json named exports Jan 13, 2025
@IWANABETHATGUY IWANABETHATGUY added this pull request to the merge queue Jan 13, 2025
Merged via the queue into rolldown:main with commit b8f0e19 Jan 13, 2025
27 checks passed
@hi-ogawa hi-ogawa deleted the fix-json-parse-pure branch January 13, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants