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

Does this plugin work? #358

Open
helgardferreira opened this issue Jun 8, 2024 · 1 comment
Open

Does this plugin work? #358

helgardferreira opened this issue Jun 8, 2024 · 1 comment

Comments

@helgardferreira
Copy link

helgardferreira commented Jun 8, 2024

I'm getting false flags when I'm using converters on @property for both no-incompatible-property-type and no-incompatible-type-binding. I can remove the no-incompatible-property-type error by not using ComplexAttributeConverter but no matter what I do no-incompatible-type-binding is a PITA.

I've seen multiple issues of this on the repo and at this stage I'm wondering why no-incompatible-type-binding even exists? no-incompatible-property-type makes some sense to prevent developers from using @property for complex types without an accompanying converter but no-incompatible-type-binding makes essentially zero sense since attributes from the DOM perspective are always string??

Either the type system for no-incompatible-type-binding needs to be improved or just flat-out remove the rule. At this point it seems I either have to litter my codebase with @ts-ignore, @ts-expect-error, or just fork this project and remove the superfluous rule.

@xiaohk
Copy link

xiaohk commented Oct 25, 2024

Same issue here. Did you manage to solve the issue? @helgardferreira

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants