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

Editing .csx: XML Docs of own libraries are not displayed in Intellisense popups #1026

Closed
filipw opened this issue Nov 13, 2017 · 3 comments · Fixed by #1027
Closed

Editing .csx: XML Docs of own libraries are not displayed in Intellisense popups #1026

filipw opened this issue Nov 13, 2017 · 3 comments · Fixed by #1027

Comments

@filipw
Copy link
Member

filipw commented Nov 13, 2017

From @KarloX2 on November 13, 2017 5:44

Environment data

dotnet --info output:
.NET Command Line Tools (1.0.0-preview2-003131)

Product Information:
Version: 1.0.0-preview2-003131
Commit SHA-1 hash: 635cf40e58

Runtime Environment:
OS Name: Windows
OS Version: 10.0.15063
OS Platform: Windows
RID: win10-x64

VS Code version: 1.18.0
C# Extension version: 1.13.0

Steps to reproduce

Create a .csx files in VS Code and put the line

#r "C:\Some\Folder\MyLibrary.dll"

in the first line. Make sure that, in the same folder, a .xml documentation file (crated by Visual Studio along with the .dll) exists. Assume there's a class MyClass defined in the dll and documented in the .xml. Try to declare a variable of that class type and observe that the documentation is not being displayed in the IntelliSense popup.

MyClass myVariable; //<<<--- IntelliSense will show signatures but not the docs.

Expected behavior

Member documentation shall be included in IntelliSense popups.

Actual behavior

No member documentation is displayed.

Copied from original issue: dotnet/vscode-csharp#1858

@filipw
Copy link
Member Author

filipw commented Nov 13, 2017

I had a look at this, and it is caused by the fact that we moved to use Roslyn's RuntimeMetadataReferenceResolver for resolving #r, which skips XML docs here.

Using RuntimeMetadataReferenceResolver gives us many other benefits so we should continue doing so, but we should work around the XML loading or send an upstream PR to Roslyn to fix this.

filipw added a commit to filipw/omnisharp-roslyn that referenced this issue Nov 13, 2017
filipw added a commit to filipw/omnisharp-roslyn that referenced this issue Nov 13, 2017
@KarloX2
Copy link

KarloX2 commented Dec 30, 2017

@filipw: Is this fix already available in a current release? It seems not to work in 1.13.1.
Thanks

@filipw
Copy link
Member Author

filipw commented Dec 30, 2017

not yet, it's part of OmniSharp Roslyn 1.28.0 (https://github.com/OmniSharp/omnisharp-roslyn/releases/tag/v1.28.0) but C# for VS Code 1.13.1 is based on OmniSharp Roslyn 1.26.3.

Actually the timing was a bit unfortunate, this fix was merged on 14 Nov, but 1.13.1 came out on 13 Nov.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants