-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
Warn me if I use @observable without extends Object with ObservableMixin
#12357
Comments
=), this is basically what this TODO is all about: I initially had more warnings, but it would have had false negatives (e.g. if you extend from Observable indirectly). So for now we only report a warning if there isn't a base class or mixin. We could fix that if we did more work with the analyzer (resolving types and such). |
cc @sigmundch. |
Added this to the M8 milestone. |
if anything I think we're more likely to go the dirty-checking fallback route: Added NotPlanned label. |
It would help certain users such as myself to be warned if they (me) use @observable without also putting
extends Object with ObservableMixin
The text was updated successfully, but these errors were encountered: