sveltejs / svelte

Cybernetically enhanced web apps
https://svelte.dev
MIT License
76.98k stars 4k forks source link

fix: skip deriveds for props with known safe calls #11595

Closed Rich-Harris closed 2 weeks ago

Rich-Harris commented 2 weeks ago

Very small thing but I noticed while working on #11571 and #11574 that we treat certain function calls as 'safe' for the purposes of determining whether to wrap props in deriveds, but we only use that information for spread attributes, because we put the parentheses in the wrong place.

No test because it's finicky and extremely minor

Before submitting the PR, please make sure you do the following

Tests and linting

changeset-bot[bot] commented 2 weeks ago

🦋 Changeset detected

Latest commit: 0eea150843444917a289b583dc34c49948387181

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