Skip to content

VS Code language service is broken in 1.1.0 due to unessecary package #6536

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

Closed
cyrilletuzi opened this issue Jun 1, 2017 · 6 comments
Closed
Assignees
Labels
effort1: easy (hours) P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent

Comments

@cyrilletuzi
Copy link
Contributor

cyrilletuzi commented Jun 1, 2017

Bug Report or Feature Request (mark with an x)

- [x] bug report 

Versions.

cli 1.1.0
VS Code Angular Language Service 0.1.3

Repro steps.

  1. Use VS Code with official Angular Language Service plugin installed.
  2. Create a project with CLI.
  3. Completion and error checking in HTML external templates doesn't work.
  4. npm uninstall @angular/language-service
  5. Restart VS Code.
  6. Now it works.

Desired functionality.

VS Code Angular Language Service plugin seems to use :

  • it's own install of @angular/language-service if it's not present in node_modules
  • if present, it uses the project dependency, and unfortunately, the plugin is not always up to date with the last release of @angular/language-service.

I see no use to pre-install @angular/language-service as the developer won't use it directly, and plugins should manage their own dependencies to be sure to use the good version.

So I think commit bb99fac should be reverted.

I've open the same issue in VS Code Angular Language Service project too : angular/vscode-ng-language-service#102

@cyrilletuzi
Copy link
Contributor Author

Note : adding the @angular/language-service by default will be great when we'll be able to just add in tsconfig.json :

"plugins": [{ "name": "@angular/language-service" }]

Then Angular language service will be working out of the box, with no plugin needed.

But it's not ready yet, as it only works with inline templates for now (external templates support is waiting for a new TypeScript feature).

@filipesilva filipesilva added the needs: investigation Requires some digging to determine if action is needed label Jun 6, 2017
@filipesilva
Copy link
Contributor

@sumitarora can you take a look?

@sumitarora
Copy link
Contributor

@filipesilva I can confirm it's not working as expected for external templates. We can wait for the plugin to fix the issue as it seems to be an issue with VS code only.

@filipesilva filipesilva added blocked effort1: easy (hours) P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent and removed needs: investigation Requires some digging to determine if action is needed labels Jun 12, 2017
@lemontreelxs
Copy link

这个玩意儿,不能用好久了,我看了很多的讨论,都没什么结果,总之就是一句话,等!!!!!我靠

@cyrilletuzi
Copy link
Contributor Author

Closing, as it seems to be OK with last versions (even if the issue in angular/vscode-ng-language-service#102 wasn't closed).

@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Sep 7, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
effort1: easy (hours) P3 An issue that is relevant to core functions, but does not impede progress. Important, but not urgent
Projects
None yet
Development

No branches or pull requests

4 participants