You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Please describe what the rule should do:
Currently using a MaybeRef without using unref wouldn't lead to an eslint error:
constmaybeRef: MaybeRef<boolean>=ref(false)if(maybeRef){// would be always true in this case but not catched with @typescript-eslint/no-unnecessary-condition
...
}
It should be enforced using unref in this case to prevent false positives in conditions.
Name of rule tbd.
What category should the rule belong to?
[ ] Enforces code style (layout)
[x] Warns about a potential error (problem)
[ ] Suggests an alternate way of doing something (suggestion)
[ ] Other (please specify:)
The text was updated successfully, but these errors were encountered:
This would require type information, so it would only be possible when typescript-eslint's parser is used.
Other patterns that should be reported:
(maybeRef ? x : y)!maybeRefmaybeRef||xmaybeRef&&ymaybeRef??xx||maybeRefx&&maybeRefx??maybeRefmaybeRef==xmaybeRef!=xmaybeRef===xmaybeRef!==xx==maybeRefx!=maybeRefx===maybeRefx!==maybeRefBoolean(maybeRef)String(maybeRef)
As a name, I'd suggest vue/require-mayberef-unwrap
Please describe what the rule should do:
Currently using a MaybeRef without using
unref
wouldn't lead to an eslint error:It should be enforced using unref in this case to prevent false positives in conditions.
Name of rule tbd.
What category should the rule belong to?
[ ] Enforces code style (layout)
[x] Warns about a potential error (problem)
[ ] Suggests an alternate way of doing something (suggestion)
[ ] Other (please specify:)
The text was updated successfully, but these errors were encountered: