chore: simplify props
#16270
Pull request
Merged
R
Rich-Harris Jul 1, 2025, 2:16 AMWhile looking into #16263 I realised that we can drastically simplify the prop implementation now that deriveds are writable. Some of the code in red is pretty hard to understand, but deleting it causes no tests to fail. Just to be safe, I added a changeset so that this creates a new version that we can roll back from if it causes any issues.
Before submitting the PR, please make sure you do the following
- It's really useful if your PR references an issue where it is discussed ahead of time. In many cases, features are absent for a reason. For large changes, please create an RFC: https://github.com/sveltejs/rfcs
- Prefix your PR title with
feat:,fix:,chore:, ordocs:. - This message body should clearly illustrate what problems it solves.
- Ideally, include a test that fails without this PR but passes with it.
- If this PR changes code within
packages/svelte/src, add a changeset (npx changeset).
Tests and linting
- Run the tests with
pnpm testand lint the project withpnpm lint
Info
Merged at Jul 1, 2025, 6:20 PM
Merged by Rich-Harris
Assignees None
Reviewers None
Labels None
Milestone None
C
changeset-bot[bot] Jul 1, 2025, 2:16 AM🦋 Changeset detected
Latest commit: 0ef587a
The changes in this PR will be included in the next version bump.
This PR includes changesets to release 1 package
| Name | Type |
|---|---|
| svelte | Patch |
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
S
svelte-docs-bot[bot] Jul 1, 2025, 2:18 AM G
github-actions[bot] Jul 1, 2025, 2:19 AMpnpm add https://pkg.pr.new/svelte@16270Pull request opened Jul 1, 2025, 2:16 AM
Pull request merged Jul 1, 2025, 6:20 PM
Total -122 +73