Skip to content

test: Vite integration tests#899

Draft
timfish wants to merge 1 commit intomainfrom
timfish/test/vite-integration-tests
Draft

test: Vite integration tests#899
timfish wants to merge 1 commit intomainfrom
timfish/test/vite-integration-tests

Conversation

@timfish
Copy link
Collaborator

@timfish timfish commented Mar 22, 2026

This PR adds new more complete integration tests for Vite 4, 7 and 8. v4 will be the oldest supported version and v7 is the last major with Rollup. v8 is latest using Rolldown.

To get all the installed bundlers and dependencies working together I needed to update the Rolldown rc version which then changed the Rolldown snapshots. It appears the latest rc disabled some minification with the default configuration.

I've fixed all the bundler versions used to build the modules and for the integration tests. Without this, adding any bundler dependency can result in the rollup/rolldown changing/failing. This shouldn't happen so maybe ancient yarn broken resolution 🤷‍♂️

@github-actions
Copy link

github-actions bot commented Mar 22, 2026

Semver Impact of This PR

🟢 Patch (bug fixes)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • Use crypto.randomUUID rather than uuid by timfish in #892

Internal Changes 🔧

  • Vite integration tests by timfish in #899
  • Rollup integration tests by timfish in #897
  • New integration tests by timfish in #896
  • Remove lerna by timfish in #895
  • Migrate to Vitest by timfish in #894

🤖 This preview updates automatically when you update the PR.

@timfish timfish force-pushed the timfish/test/vite-integration-tests branch 3 times, most recently from f90d384 to 502c808 Compare March 23, 2026 01:17
@timfish timfish force-pushed the timfish/test/vite-integration-tests branch from 502c808 to 85408dc Compare March 23, 2026 01:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant