Skip to content

Conversation

@sapphi-red
Copy link
Contributor

@sapphi-red sapphi-red commented Apr 22, 2025

Changes

Upgrades Vite to 6.3 that includes the revised version of cssScopeTo (vitejs/vite#19678) that aligns with astro's cssScopeTo implementation and reapplies #13347.

Testing

Existing tests should pass. There's added test in the past from #10291. If CI fails, maybe there's a bug in Vite.

Docs

N/A Shouldn't require a changeset as it's an internal refactor (the changeset for upgrading Vite is included).

@changeset-bot
Copy link

changeset-bot bot commented Apr 22, 2025

🦋 Changeset detected

Latest commit: 08373b6

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

@github-actions github-actions bot added pkg: svelte Related to Svelte (scope) pkg: vue Related to Vue (scope) pkg: react Related to React (scope) pkg: preact Related to Preact (scope) pkg: solid Related to Solid (scope) pkg: integration Related to any renderer integration (scope) pkg: astro Related to the core `astro` package (scope) labels Apr 22, 2025
@ematipico ematipico added the pr preview Apply this label to a PR to generate a preview release label Apr 22, 2025
@pkg-pr-new
Copy link

pkg-pr-new bot commented Apr 22, 2025

astro

npm i https://pkg.pr.new/astro@13668

@astrojs/cloudflare

npm i https://pkg.pr.new/@astrojs/cloudflare@13668

@astrojs/netlify

npm i https://pkg.pr.new/@astrojs/netlify@13668

@astrojs/node

npm i https://pkg.pr.new/@astrojs/node@13668

@astrojs/vercel

npm i https://pkg.pr.new/@astrojs/vercel@13668

commit: 016ccc3

@sapphi-red sapphi-red marked this pull request as ready for review April 22, 2025 09:51
@codspeed-hq
Copy link

codspeed-hq bot commented Apr 22, 2025

CodSpeed Performance Report

Merging #13668 will not alter performance

Comparing sapphi-red:vite-css-scope-to (08373b6) with main (debd69d)

Summary

✅ 6 untouched benchmarks

@ematipico
Copy link
Member

I just tested this on the Biome website, and I couldn't find any regressions

@github-actions github-actions bot removed pkg: svelte Related to Svelte (scope) pkg: vue Related to Vue (scope) pkg: react Related to React (scope) pkg: preact Related to Preact (scope) pkg: solid Related to Solid (scope) pkg: integration Related to any renderer integration (scope) labels May 14, 2025
@sapphi-red
Copy link
Contributor Author

I've resolved the conflicts.

@ematipico
Copy link
Member

ematipico commented May 14, 2025

@sapphi-red can you please provide a changeset? Even if it's an internal change, we want to track it so if there are some regressions, there's a changelog line that refers to it. A patch is enough

@sapphi-red
Copy link
Contributor Author

Done 👍

@ematipico ematipico merged commit 866285a into withastro:main May 14, 2025
16 checks passed
@astrobot-houston astrobot-houston mentioned this pull request May 14, 2025
ematipico added a commit that referenced this pull request May 16, 2025
openscript pushed a commit to openscript/astro that referenced this pull request Sep 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pkg: astro Related to the core `astro` package (scope) pr preview Apply this label to a PR to generate a preview release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants