Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(rspack_plugin_schemas): http import schema plugin #7008

Merged
merged 116 commits into from
Oct 10, 2024

Conversation

ScriptedAlchemy
Copy link
Contributor

@ScriptedAlchemy ScriptedAlchemy commented Jul 2, 2024

Summary

Implements the basic capabilities of HTTP URI Plugin

Example: https://github.com/ScriptedAlchemy/rspack-rust-api

Allows importing of esm http imports and bundling them

Checklist

  • Tests updated (or not required).
  • Documentation updated (or not required).

Base automatically changed from nmf-resolve-hook to main July 2, 2024 03:56
Copy link

netlify bot commented Jul 2, 2024

Deploy Preview for rspack ready!

Name Link
🔨 Latest commit e8bfc85
🔍 Latest deploy log https://app.netlify.com/sites/rspack/deploys/67048c94de4ff500088f42b6
😎 Deploy Preview https://deploy-preview-7008--rspack.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/rspack_http_with-resolve branch 9 times, most recently from 5a68a80 to 948e244 Compare July 8, 2024 20:28
@ScriptedAlchemy ScriptedAlchemy changed the title Feat/rspack http with resolve feat(rspack_plugin_schemas): http uri plugin Jul 8, 2024
@github-actions github-actions bot added the release: feature release: feature related release(mr only) label Jul 8, 2024
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/rspack_http_with-resolve branch 6 times, most recently from 216850f to 8decef9 Compare July 10, 2024 00:42
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/rspack_http_with-resolve branch 9 times, most recently from c650fc9 to aabf81f Compare July 23, 2024 10:15
@ScriptedAlchemy ScriptedAlchemy force-pushed the feat/rspack_http_with-resolve branch 2 times, most recently from d77760e to 201c858 Compare July 29, 2024 07:00
@hardfist
Copy link
Contributor

@ScriptedAlchemy should we expose this feature to js side?

@ScriptedAlchemy
Copy link
Contributor Author

Yes, we should, but I have never done bindings for rspack and am not sure how this is accomplished.

@hardfist
Copy link
Contributor

hardfist commented Oct 10, 2024

we can merge this first and add binding and js side test in following pr,@SyMind can you help add jsbinding for this?

@ScriptedAlchemy
Copy link
Contributor Author

@SyMind, would you tag me in the PR so I can read and understand it end-to-end? Then, next time, I'll try to create bindings on my own. 🙃

@ScriptedAlchemy
Copy link
Contributor Author

Yeah that's fine. My current use cases is with rust projects so I am not blocked by js. But user will want it for sure

@hardfist hardfist merged commit 598d44e into main Oct 10, 2024
29 checks passed
@hardfist hardfist deleted the feat/rspack_http_with-resolve branch October 10, 2024 23:57
@SyMind
Copy link
Member

SyMind commented Oct 11, 2024

@ScriptedAlchemy @hardfist I got it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
release: feature release: feature related release(mr only)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants