-
-
Notifications
You must be signed in to change notification settings - Fork 2.5k
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
Stabilize inline stylesheets #7180
Conversation
🦋 Changeset detectedLatest commit: c2af343 The changes in this PR will be included in the next version bump. 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 |
@@ -0,0 +1,24 @@ | |||
--- | |||
'astro': major |
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.
We probably won't release a major for another couple of months, we're going to time it with the next Vite major.
So for now I think this should be a minor
and continue to have never
as the default. In 3.0 we can switch to auto
.
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.
You're right, I forgot what major means.
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.
We want this to be a non-breaking change for now, just out of experimental.
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.
Docs are good! Just update the version to reflect when it was stable, I think is the safer option?
4f336e5
to
7de8901
Compare
Co-authored-by: Sarah Rainsberger <sarah@rainsberger.ca>
7de8901
to
c2af343
Compare
Changes
Stabilize
AstroConfig.build.inlineStylesheets
.Testing
Existing ones should pass.
Docs
I think Configuration reference as-is would be sufficient.