Skip to content
This repository has been archived by the owner on May 17, 2024. It is now read-only.

Adds VB support for Unit Tests and WPF #390

Merged
merged 4 commits into from
Mar 27, 2021
Merged

Adds VB support for Unit Tests and WPF #390

merged 4 commits into from
Mar 27, 2021

Conversation

KSchlobohm
Copy link
Contributor

Addresses #384

Copy link
Member

@mjrousos mjrousos left a comment

Choose a reason for hiding this comment

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

Looks good; I left some comments inline.

src/MSBuild.Abstractions/ProjectItemHelpers.cs Outdated Show resolved Hide resolved
@@ -246,10 +246,12 @@ public static class MSBuildFacts
"ProjectReference"
);

public static Guid LanguageProjectTypeVisualBasic => Guid.Parse("{F184B08F-C81C-45F6-A57F-5ABD9991F28F}");
Copy link
Member

Choose a reason for hiding this comment

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

This is the guid that was labeled F# previously. What that comment wrong or do we need to swap this guid?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I believe the comment was wrong.

Copy link
Contributor

Choose a reason for hiding this comment

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

if I recall, this was pulled from a legacy F# project. Interesting!

@KSchlobohm KSchlobohm marked this pull request as ready for review March 26, 2021 19:46
@KSchlobohm
Copy link
Contributor Author

@cartermp for review

Copy link
Contributor

@cartermp cartermp left a comment

Choose a reason for hiding this comment

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

Thanks! I think this looks good. I will merge this in now and I'll publish a release next week.

@cartermp cartermp linked an issue Mar 27, 2021 that may be closed by this pull request
@cartermp cartermp merged commit c6c5f3c into dotnet:main Mar 27, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add support for Visual Basic WPF and Visual Basic Unit Test projects
3 participants