Skip to content

feat: support loading dataurl as module.#1741

Merged
hyf0 merged 17 commits intorolldown:mainfrom
7086cmd:feat/dataurl-loading
Jul 27, 2024
Merged

feat: support loading dataurl as module.#1741
hyf0 merged 17 commits intorolldown:mainfrom
7086cmd:feat/dataurl-loading

Conversation

@7086cmd
Copy link
Copy Markdown
Contributor

@7086cmd 7086cmd commented Jul 26, 2024

This PR ports functionality from Vite and enables esbuild's tests.

Please note that there is an outstanding issue referenced here that is pending a fix. However, it seems unlikely that this PR will address it, as the complexity of handling the module as an external module again in load_source.rs is considerable.

@netlify
Copy link
Copy Markdown

netlify Bot commented Jul 26, 2024

Deploy Preview for rolldown-rs canceled.

Name Link
🔨 Latest commit b6cba8e
🔍 Latest deploy log https://app.netlify.com/sites/rolldown-rs/deploys/66a536ff8b0c6500088748f2

Comment thread crates/rolldown/tests/esbuild/loader/loader_data_url_unknown_mime/artifacts.snap Outdated
Comment thread crates/rolldown_utils/src/dataurl.rs Outdated
@hyf0 hyf0 self-assigned this Jul 27, 2024
Comment thread crates/rolldown/tests/esbuild/default/auto_external/artifacts.snap
Copy link
Copy Markdown
Contributor Author

@7086cmd 7086cmd left a comment

Choose a reason for hiding this comment

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

Seems Ok now.

Comment thread crates/rolldown_utils/src/dataurl.rs Outdated
@7086cmd 7086cmd requested a review from IWANABETHATGUY July 27, 2024 15:13
Comment thread crates/rolldown_plugin/src/utils/resolve_id_with_plugins.rs Outdated
@hyf0
Copy link
Copy Markdown
Member

hyf0 commented Jul 27, 2024

Thanks for the contribution. I'm gonna make some changes before merging this PR.

@hyf0 hyf0 added this pull request to the merge queue Jul 27, 2024
Merged via the queue into rolldown:main with commit 549e1e1 Jul 27, 2024
@7086cmd 7086cmd deleted the feat/dataurl-loading branch August 7, 2024 04:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants