Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix: svelte:element with dynamic this and spread attributes throws error #9112

Merged
merged 7 commits into from
Sep 20, 2023

Conversation

teobgeno
Copy link
Contributor

@teobgeno teobgeno commented Aug 17, 2023

fixes #9092

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:, or docs:.
  • This message body should clearly illustrate what problems it solves.
  • Ideally, include a test that fails without this PR but passes with it.

Tests and linting

  • Run the tests with pnpm test and lint the project with pnpm lint

@changeset-bot
Copy link

changeset-bot bot commented Aug 17, 2023

🦋 Changeset detected

Latest commit: 1ef42c1

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

Copy link
Contributor

@gtm-nayan gtm-nayan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can you add a runtime test, something similar to the reproduction in the issue.

@teobgeno
Copy link
Contributor Author

teobgeno commented Aug 18, 2023

Just add it. Thanks for the tip.

…nt/index.js

Co-authored-by: gtmnayan <50981692+gtm-nayan@users.noreply.github.com>
Copy link
Member

@dummdidumm dummdidumm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you!

.changeset/khaki-wolves-dream.md Outdated Show resolved Hide resolved
@dummdidumm dummdidumm merged commit 228dd4d into sveltejs:master Sep 20, 2023
@github-actions github-actions bot mentioned this pull request Sep 20, 2023
kelvinsjk pushed a commit to kelvinsjk/svelte that referenced this pull request Oct 19, 2023
…ror (sveltejs#9112)

fixes sveltejs#9092

---------

Co-authored-by: gtmnayan <50981692+gtm-nayan@users.noreply.github.com>
Co-authored-by: Simon H <5968653+dummdidumm@users.noreply.github.com>
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

svelte:element with dynamic this and spread attributes
3 participants