Skip to content

chore(solidstart): Remove unused Vitest setup from e2e apps - #24789

Merged
Lms24 merged 1 commit into
chore/upgrade-vitest-4from
chore/remove-solidstart-vitest
Sep 29, 2026
Merged

Lms24 merged 1 commit into
chore/upgrade-vitest-4from
chore/remove-solidstart-vitest

Conversation

@Lms24

@Lms24 Lms24 commented Sep 28, 2026 •

Copy link
Copy Markdown
Member

The SolidStart e2e apps only run Playwright, so vitest, @vitest/ui and vitest.config.ts are unused leftovers from the Solid CLI template. Noticed while working on #24746. Didn't cause a direct dependency duplication but thought it would be good to get this cleaned up.

@Lms24
Lms24 added this pull request to stack #24790 September 28, 2026 09:56
@Lms24 Lms24 self-assigned this Sep 28, 2026
@Lms24
Lms24 marked this pull request as ready for review September 28, 2026 12:31
@Lms24
Lms24 requested review from andreiborza and s1gr1d September 28, 2026 13:55
@Lms24
Lms24 requested review from a team as code owners September 29, 2026 15:58
@Lms24
Lms24 requested review from JPeer264, isaacs, logaretm, msonnb and nicohrubec and removed request for a team September 29, 2026 15:58
@Lms24
Lms24 force-pushed the chore/remove-solidstart-vitest branch from 7f660ef to 4fce78d Compare September 29, 2026 15:58
The SolidStart e2e apps only run Playwright. `vitest`, `@vitest/ui`
and `vitest.config.ts` were leftovers from the Solid CLI template and
kept old Vitest versions around in the repo.

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@Lms24
Lms24 force-pushed the chore/remove-solidstart-vitest branch from 4fce78d to 6922e24 Compare September 29, 2026 16:23
@github-actions

Copy link
Copy Markdown
Contributor

size-limit report 📦

Path Size % Change Change
@sentry/browser 29.24 kB - -
@sentry/browser - with treeshaking flags 27.5 kB - -
@sentry/browser - with treeshaking flags tracing without tracing 27.4 kB - -
@sentry/browser (incl. Tracing) 51.15 kB - -
@sentry/browser (incl. Tracing + Span Streaming) 51.18 kB - -
@sentry/browser (incl. Tracing, Profiling) 54.18 kB - -
@sentry/browser (incl. Tracing, Replay) 90.76 kB - -
@sentry/browser (incl. Tracing, Replay) - with treeshaking flags 79.86 kB - -
@sentry/browser (incl. Tracing, Replay with Canvas) 95.46 kB - -
@sentry/browser (incl. Tracing, Replay, Feedback) 108.41 kB - -
@sentry/browser (incl. Feedback) 46.76 kB - -
@sentry/browser (incl. sendFeedback) 34.3 kB - -
@sentry/browser (incl. FeedbackAsync) 39.41 kB - -
@sentry/browser (incl. Metrics) 30.25 kB - -
@sentry/browser (incl. Logs) 30.53 kB - -
@sentry/browser (incl. Metrics & Logs) 31.2 kB - -
@sentry/react 31.08 kB - -
@sentry/react (incl. Tracing) 53.54 kB - -
@sentry/vue 36.74 kB - -
@sentry/vue (incl. Tracing) 53.7 kB - -
@sentry/svelte 29.26 kB - -
CDN Bundle 31.05 kB - -
CDN Bundle (incl. Tracing) 51.8 kB - -
CDN Bundle (incl. Logs, Metrics) 33.31 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) 53.77 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) 74.02 kB - -
CDN Bundle (incl. Tracing, Replay) 89.39 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) 91.36 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) 95.55 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) 97.53 kB - -
CDN Bundle - uncompressed 91.7 kB - -
CDN Bundle (incl. Tracing) - uncompressed 154.08 kB - -
CDN Bundle (incl. Logs, Metrics) - uncompressed 98.27 kB - -
CDN Bundle (incl. Tracing, Logs, Metrics) - uncompressed 160.04 kB - -
CDN Bundle (incl. Replay, Logs, Metrics) - uncompressed 227.84 kB - -
CDN Bundle (incl. Tracing, Replay) - uncompressed 273.81 kB - -
CDN Bundle (incl. Tracing, Replay, Logs, Metrics) - uncompressed 279.75 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback) - uncompressed 287.51 kB - -
CDN Bundle (incl. Tracing, Replay, Feedback, Logs, Metrics) - uncompressed 293.44 kB - -
@sentry/nextjs (client) 55.78 kB - -
@sentry/sveltekit (client) 51.6 kB - -
@sentry/core/server 39.99 kB - -
@sentry/core/browser 13.63 kB - -
@sentry/node 142.24 kB +0.01% +7 B 🔺
@sentry/node/import (ESM hook with diagnostics-channel injection) 82.88 kB - -
@sentry/node - without tracing 91.25 kB +0.02% +12 B 🔺
@sentry/node - without channel injection 120.64 kB +0.01% +6 B 🔺
@sentry/aws-serverless 99.52 kB +0.01% +6 B 🔺
@sentry/cloudflare (withSentry) - minified 206.64 kB - -
@sentry/cloudflare (withSentry) 514.04 kB - -

View base workflow run

@Lms24
Lms24 merged commit 68b6f79 into develop Sep 29, 2026
47 checks passed
@Lms24
Lms24 deleted the chore/remove-solidstart-vitest branch September 29, 2026 16:38
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.

2 participants