-
-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Support custom svelte compiler options in @astrojs/svelte #3181
Conversation
🦋 Changeset detectedLatest commit: 730418d The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
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.
Svexcellent updates, README, and tests! Just want to remove those temporary scripts, but otherwise LGIP
…3181) * updating svelte integration to allow custom user config * test: adding a test to verify that svelte options are piped through the integration * updating the README with docs on overridding svelte options * chore: adding changeset * fix: copy/paste bug in test validation * removing temporary debug scripts
Changes
Similar to #3143, this updates the Svelte integration to allow users to override most of the Svelte config.
emitCss
,dev
, andhydratable
are always managed by the Astro integration, everything else can be overriddenTesting
Updated the
svelte-component
test to verify that overriding theextensions
option works as expected in the Svelte compilerDocs
README updated with documentation on providing custom Svelte compiler option.