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
Update plugin to use Angular 6 #264
Comments
Maybe duplicate of #256 |
I did reference that issue, but I think this should remain open as its own thing because a) it might be possible to resolve #256 without switching to Angular 6 and b) it might be possible to build against Angular 6 without actually resolving the template-scope problem I give as an example. I would like to make sure my specific problem (incorrect redlining in templates) actually gets resolved. |
This plugin is basically unusable for us right now since we take advantage of TS 2.7 features like
Angular 6 has been out for 2 months now - it would be nice to see this plugin updated. |
Is work even happening on this plugin? I see that the last commit was over 6 months ago. |
@JitinDhillon Seems like this has been abandoned. Unfortunately I don't know of any maintained forks or alternatives. |
Real shame that this has been abandoned. |
@chuckjaz @DBosley is it possible to get some information regarding the Angular team's intention to keep this project alive? There are lots of glaring open issues at the moment and it seems like this project is abandoned. If you guys are no longer maintaining this, is it possible to assign someone else from the Angular team? Edit: just noticed that @chuckjaz hasn't been active on Github for almost a year, so just to try and poke some other Angular members: @IgorMinar @vicb @petebacondarwin ? |
Hey guys, I have just made a PR to the master branch in order to support the latest Angular v6. (#276 ). I need someone to help me to take a look at those changes. |
v0.1.10 is out with angular-language-service 6.1 and TS 2.7, thanks @kyliau |
I still get the "does not contain such a member" error for |
Yeah, as far as I can tell the improved scoping for ngIf that's supposed to come with the new language service, isn't working. Other kinds of narrowing don't work either. For example, I have a class with a method |
Closing this, since as @zacnomore mentioned, the latest version is now using |
This issue has been automatically locked due to inactivity. Read more about our automatic conversation locking policy. This action has been performed automatically by a bot. |
Plugin is still built against Angular 5, should be updated to Angular 6. Might resolve #256 , and this comment on an Angular issue suggests that it's necessary for the new type checker to show up (?), which would make
stop red-lining
foo.length
whenstrictNullChecks
is turned on.As an aside, if that comment is mistaken and that edge-case still isn't handled even when built against latest Angular, please let me know and I'll open a separate issue to make sure it gets tracked. I thought this was what #71 was talking about but it's a separate case. I'm hoping that this is an easy fix, and will also cover
*ngIf="foo && (foo.length > 2)"
red-liningfoo.length
(even though the and-operator should remove null from foo's type before evaluating).The text was updated successfully, but these errors were encountered: