Closed Evangelink closed 7 months ago
Let's review the following points before we ship the feature complete preview:
is this https://github.com/microsoft/testfx/blob/main/src/Package/MSTest.Sdk/Sdk/Runner/ClassicEngine.targets#L20 needed? It seems we could do without it.
make it possible to opt-out from analyzers (e.g. https://github.com/microsoft/testfx/blob/main/src/Package/MSTest.Sdk/Sdk/Runner/ClassicEngine.targets#L61) <EnableMSTestAnalyzers>
<EnableMSTestAnalyzers>
review engine names: use the standard native aot knob used by the runtime. <PublishAot>true</PublishAot>
<PublishAot>true</PublishAot>
AB#1992497
✅ Successfully linked to Azure Boards work item(s):
Unassigning myself as my contribution here is done, looking forward for the PR @MarcoRossignoli :)
Let's review the following points before we ship the feature complete preview:
is this https://github.com/microsoft/testfx/blob/main/src/Package/MSTest.Sdk/Sdk/Runner/ClassicEngine.targets#L20 needed? It seems we could do without it.
make it possible to opt-out from analyzers (e.g. https://github.com/microsoft/testfx/blob/main/src/Package/MSTest.Sdk/Sdk/Runner/ClassicEngine.targets#L61)
<EnableMSTestAnalyzers>
review engine names: use the standard native aot knob used by the runtime.
<PublishAot>true</PublishAot>
AB#1992497