-
Notifications
You must be signed in to change notification settings - Fork 130
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(plugin-svelte): support pass preprocess options #1185
feat(plugin-svelte): support pass preprocess options #1185
Conversation
✅ Deploy Preview for rsbuild ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
* | ||
* See https://github.com/sveltejs/svelte-preprocess | ||
*/ | ||
preprocessOptions?: AutoPreprocessOptions; |
There was a problem hiding this 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 an unit test for this new option?
see: packages/plugin-svelte/tests/index.test.ts
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
done :)
but the snapshot was not serialized as expected
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
haha, maybe we can improve the snapshot in the future
3bb913b
to
eb07d3b
Compare
Thanks! ❤️ Can you submit another PR to update the document? |
Summary
Related Links
Checklist