Skip to content

Default emitCssInSsr if Remix plugin is present#1239

Merged
markdalgleish merged 1 commit intomasterfrom
remix-emit-css-in-ssr
Nov 20, 2023
Merged

Default emitCssInSsr if Remix plugin is present#1239
markdalgleish merged 1 commit intomasterfrom
remix-emit-css-in-ssr

Conversation

@markdalgleish
Copy link
Contributor

This flag needed to be enabled in the Remix Vite plugin test suite in order for it to pass when using VE, so this PR adds Remix to the ever growing list of frameworks that need this feature enabled. This seems to be a sign that this should maybe be the default, but since that's likely to be considered a breaking change, it makes sense that this could go through as a non-breaking fix in the meantime.

@markdalgleish markdalgleish added the vite Issue related to vite label Nov 20, 2023
@changeset-bot
Copy link

changeset-bot bot commented Nov 20, 2023

🦋 Changeset detected

Latest commit: 688727a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package
Name Type
@vanilla-extract/vite-plugin Patch

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

vite Issue related to vite

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants