Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

feat(template-compiler): warning for slot attribute not at the top level #4711

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

cardoso
Copy link
Contributor

@cardoso cardoso commented Oct 26, 2024

Details

Rough implementation for now to discuss. Leaving as draft.

TODO:

  • Decide on all valid cases
  • Add tests for invalid and valid cases
  • Improve the warning
  • Clean up implementation

Closes #4703

Does this pull request introduce a breaking change?

  • 😮‍💨 No, it does not introduce a breaking change.

Does this pull request introduce an observable change?

  • 🔬 Yes, it does include an observable change.

A new warning when compiling a template with an element that has a slot attribute which is likely to be ignored.

GUS work item

@cardoso cardoso force-pushed the slot-top-level-warning branch 3 times, most recently from 2796e28 to 3d7769a Compare October 26, 2024 02:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Show compiler warning/error for slot attribute not at the top level
1 participant