Skip to content

Conversation

@BillWagner
Copy link
Member

Fixes #50734

The sample code was missing the needed "Imports" statement, and the declaration was commented out.

Also, the project needed to import the type library.

Fixes dotnet#50734

The sample code was missing the needed "Imports" statement, and the declaration was commented out.

ALso, the project needed to import the type library.
Copilot AI review requested due to automatic review settings January 29, 2026 21:27
@BillWagner BillWagner requested a review from a team as a code owner January 29, 2026 21:27
@dotnetrepoman dotnetrepoman bot added this to the January 2026 milestone Jan 29, 2026
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Fixes the VB COM interop documentation sample so the snippet includes the required Imports statement and restores the previously commented-out declaration, and updates the sample project to add the referenced COM type library.

Changes:

  • Added a COM type library reference for INKEDLib to the sample VB project.
  • Updated the extracted snippet to include Imports INKEDLib and uncommented the IInkCursor field declaration.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
samples/snippets/visualbasic/VS_Snippets_VBCSharp/VbVbalrInterop/VB/vbvbalrinterop.vbproj Adds a COM type library reference to support the COM interop snippet.
samples/snippets/visualbasic/VS_Snippets_VBCSharp/VbVbalrInterop/VB/Class1.vb Fixes snippet content by adding the missing Imports and restoring the declaration.

@BillWagner BillWagner enabled auto-merge (squash) January 29, 2026 21:38
@BillWagner BillWagner merged commit 51e88a5 into dotnet:main Jan 30, 2026
9 checks passed
@BillWagner BillWagner deleted the fix-vb-sample-include branch January 30, 2026 16:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[UUF] Code snippet is missing info

2 participants