Skip to content

chore: faster CI - #16842

Open
Rich-Harris wants to merge 3 commits into
version-3from
faster-tests
Open

chore: faster CI#16842
Rich-Harris wants to merge 3 commits into
version-3from
faster-tests

Conversation

@Rich-Harris

Copy link
Copy Markdown
Member

I have a hunch that there are several things we can be doing to improve the time it takes to run CI. I asked an agent to find the lowest-hanging fruit and this is what it came up with — mostly enabling more parallelism, though it did also remove Node 22 from the e2e test matrix, which is possibly dangerous. Let's see how much this buys us

@pkg-svelte-dev

pkg-svelte-dev Bot commented Aug 18, 2026

Copy link
Copy Markdown

Install the latest version of @sveltejs/kit from 1a86305:

pnpm add https://pkg.svelte.dev/@sveltejs/kit/c/1a8630501fa7351e842e69d5becf49c772290f21

Open in pkg.svelte.dev: https://pkg.svelte.dev/repos/kit/pr/16842

@changeset-bot

changeset-bot Bot commented Aug 18, 2026

Copy link
Copy Markdown

⚠️ No Changeset found

Latest commit: 1a86305

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@svelte-docs-bot

Copy link
Copy Markdown

@teemingc

teemingc commented Aug 18, 2026

Copy link
Copy Markdown
Member

There's also a setting for '50%' workers, etc. to adjust to the number of cores the runner has rather than a strict number

I also suspect there may be some tests we're running in multiple suites which we can deduplicate but I haven't looked deeply into it.

Removing Node 22 for test-kit probably frees up a good number of runners. Those are currently the longest tests

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.

3 participants