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

Error underlining fails in composite project setup #35587

Closed
NoelAbrahams opened this issue Dec 9, 2019 · 5 comments
Closed

Error underlining fails in composite project setup #35587

NoelAbrahams opened this issue Dec 9, 2019 · 5 comments
Assignees
Labels
Needs Investigation This issue needs a team member to investigate its status.

Comments

@NoelAbrahams
Copy link

TypeScript Version: 3.7.2
Visual Studio Version: VS 2019 Community (16.4)

Search Terms:
error underlining

Repro Steps

  • Project A contains 'library' code in file aaa.ts (Project has 'composite' set to true).
  • Project B references A and consumes a interface Foo declared in aaa.ts
  • Ensure all files in B are closed.
  • Open file aaa.ts.
  • Add a new method to interface Foo (which would cause breakage in B).
  • Rebuild B and check the output window for the expected error.
  • In the output window double click on the error line to see the error.
  • This will open the offending file in B.

Expected behavior:
The file in B should have the error underlined

Actual behavior:
There is no error underlining.

Other points to note:

  • Restarting Visual Studio causes the error underlining to show up correctly.
  • If the offending file in Project B is already open then the error underlining works correctly.

There is no way to reproduce this as the error occurs only in a very large project set up. However, we have captured language service logs according to the instructions. Please provide a contact email to send through the logs. I cannot post them here as there is potentially sensitive information.

Usual spiel on why this should be fixed urgently
We are basically operating in the dark without error underlining.

@sheetalkamat
Copy link
Member

Please email me the logs, Email id is shkamat at microsoft dot com

@NoelAbrahams
Copy link
Author

@sheetalkamat thanks. I've emailed the logs. FYI I've just upgraded to TS 3.7.3 and the bug is still there.

@NoelAbrahams
Copy link
Author

@sheetalkamat thanks for the email. I need to look into this and will try to get back tomorrow.

@sheetalkamat
Copy link
Member

Adding @PranavSenthilnathan and @uniqueiniquity to investigate the VS side for this. I have forwarded your logs to them to take a look at it.

@sheetalkamat sheetalkamat removed their assignment Dec 17, 2019
@sheetalkamat sheetalkamat added the Needs Investigation This issue needs a team member to investigate its status. label Dec 17, 2019
@uniqueiniquity
Copy link
Contributor

uniqueiniquity commented Jan 28, 2020

Changes to resolve this in TS server have been merged in #36383 and changes to take advantage of the TS-Server-side changes will be included in a future version of Visual Studio 2019. Thank you so much for reporting and helping us to investigate this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Investigation This issue needs a team member to investigate its status.
Projects
None yet
Development

No branches or pull requests

4 participants