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

Default emitCssInSsr if Remix plugin is present #1239

Merged
merged 1 commit into from
Nov 20, 2023

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
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

@markdalgleish markdalgleish merged commit 1791df2 into master Nov 20, 2023
11 checks passed
@markdalgleish markdalgleish deleted the remix-emit-css-in-ssr branch November 20, 2023 01:37
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