Update to .NET 10 SDK and MTP#1141
Update to .NET 10 SDK and MTP#1141Youssef1313 wants to merge 7 commits intoCommunityToolkit:mainfrom
Conversation
|
Blocked on dotnet/sdk#51398 |
Sergio0694
left a comment
There was a problem hiding this comment.
Can you also pls rebase on main to remove the .NET SDK changes? 🙂
| <RunSettings> | ||
| <MSTest> | ||
| <MaxCpuCount>0</MaxCpuCount> | ||
| <Parallelize> |
There was a problem hiding this comment.
Do we still need this?
dotnet/build/Community.Toolkit.Common.targets
Lines 19 to 22 in eb25f10
Or can we remove it in this PR if it's redundant now?
There was a problem hiding this comment.
Interesting. Only one of these are needed, assuming Community.Toolkit.Common.targets is imported for all test projects in the repo.
I would prefer the one in targets and delete the runsettings. Having the assembly attribute provides:
- More consistent experience.
- Hint to MSTest analyzer MSTEST0001 that parallelization is enabled, which will otherwise show a by-design false positive.
| <RepositoryDirectory>$(MSBuildThisFileDirectory)</RepositoryDirectory> | ||
| <BuildToolsDirectory>$(RepositoryDirectory)build\</BuildToolsDirectory> | ||
|
|
||
| <EnableMSTestRunner>true</EnableMSTestRunner> |
There was a problem hiding this comment.
Can you add a comment to explain what this is and what it does? Or a link to docs if there's any.
|
|
||
| # Run tests | ||
| - name: Test solution | ||
| run: dotnet test --no-build -c ${{matrix.configuration}} -l "console;verbosity=detailed" |
There was a problem hiding this comment.
Is this because the verbosity is now detailed by default, or something else?
There was a problem hiding this comment.
The whole UX for MTP is different, and there is no -l (logger). I'm not sure what piece of info was originally intended to be looked at with detailed verbosity, maybe @nohwnd might know what VSTest exactly does under each verbosity, but anyways, I guess if there is something you don't like with the new output, we would love to know to add more customizations and/or having better defaults.
There was a problem hiding this comment.
If detailed shows passed tests, then --output detailed would be kinda the equivalent here.
Closes #<ISSUE_NUMBER>
PR Checklist
Other information