Skip to content

Conversation

Rich-Harris
Copy link
Member

realised we can simplify this a bit while looking into #16612 etc

Copy link

changeset-bot bot commented Aug 15, 2025

⚠️ No Changeset found

Latest commit: 0ea096e

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

@Rich-Harris Rich-Harris changed the title update/simplify test chore: update/simplify test Aug 15, 2025
Copy link
Contributor

Playground

pnpm add https://pkg.pr.new/svelte@16630

@dummdidumm
Copy link
Member

you should be able to do the same for the next test that is the same except for using $effect.pre instead of $effect

@Rich-Harris
Copy link
Member Author

good catch, done

@dummdidumm dummdidumm merged commit 7b2d774 into main Aug 15, 2025
16 checks passed
@dummdidumm dummdidumm deleted the simplify-effect-order-test branch August 15, 2025 21:09
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