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

Make MSTest.Analyzers a dependency of MSTest.TestFramework #4325

Closed
Evangelink opened this issue Dec 12, 2024 · 1 comment · Fixed by #4351
Closed

Make MSTest.Analyzers a dependency of MSTest.TestFramework #4325

Evangelink opened this issue Dec 12, 2024 · 1 comment · Fixed by #4351

Comments

@Evangelink
Copy link
Member

Summary

Make MSTest.Analyzers a dependency of MSTest.TestFramework.

Background and Motivation

MSTest.Analyzers currently exists but it is nearly not downloaded compared to the framework and I think we should have a 1:1 ratio as it brings only added value and no defaults. I have checked there is no impact for F# users.

Proposed Feature

Make MSTest.Analyzers a dependency of MSTest.TestFramework.

Before:

<PackageReference Include="MSTest.TestFramework" />
<PackageReference Include="MSTest.Analyzers" />

After:

<PackageReference Include="MSTest.TestFramework" />

For users relying on MSTest metapackage or MSTest.Sdk it would be a transparent update.

Alternative Designs

We could avoid extra package deps and directly include the analyzer dlls into MSTest.TestFramework but this would cause confusion and potential errors for users who are using MSTest.Analyzers today. It would also make it out of sync in term of versioning.

@Evangelink
Copy link
Member Author

Closing as the idea is mainly not liked internally.

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.

1 participant