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

All analyzers are implemented for C#/VB but will only work for C# due to the way they are packaged #4209

Closed
Youssef1313 opened this issue Dec 2, 2024 · 3 comments · Fixed by #4224

Comments

@Youssef1313
Copy link
Member

<TfmSpecificPackageFile Include="$(OutputPath)\MSTest.Analyzers.dll" PackagePath="analyzers/dotnet/cs" />
<TfmSpecificPackageFile Include="$(OutputPath)\MSTest.Analyzers.CodeFixes.dll" PackagePath="analyzers/dotnet/cs" />

We are packaging the dlls in C#-specific directory.

This issue is very separate and different from #4031

@Evangelink I'm not sure how important this one is.

@Evangelink
Copy link
Member

Oh well, I haven't tested that case so yup my bad!

@nohwnd
Copy link
Member

nohwnd commented Dec 2, 2024

is there a typo in the header? will only work for C# rather?

@Youssef1313 Youssef1313 changed the title All analyzers are implemented for C#/VB but will only work for VB due to the way they are packaged All analyzers are implemented for C#/VB but will only work for C# due to the way they are packaged Dec 2, 2024
@Youssef1313
Copy link
Member Author

Yup 😄

Fixed.

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