search

sveltejs / svelte

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

make invalid `<svelte:element this>` a warning instead of an error #11641

Closed Rich-Harris closed 2 weeks ago

Rich-Harris commented 2 weeks ago

This is an alternative to #11454. I think it's a very bad idea. Fixes #11398.

changeset-bot[bot] commented 2 weeks ago

⚠️ No Changeset found

Latest commit: 961c87df0205cb117b8e835f4927deb74ca3593d

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

dummdidumm commented 2 weeks ago

I vote for removing the code around special-casing this-strings like in #11454 but only give a warning.