Report "preview" more classy

richlander commented 1 year ago

It should be no surprise that I install a LOT of previews.

The following is really poor UX.

Warning is ever-present. The only "fixer" for this is to uninstall the preview. That seems odd.
The first line presented is a version, but isn't the version at issue and doesn't state that it is a preview.
No other versions (beyond file paths) are presented

More generally, why are we showing the MSBuild "logo" at all? We're not doing that for NuGet. We should go "no logo" for all the components we use and print the .NET version and feel free to put "preview" in marquee. At that point, we can hopefully get rid of this warning.

$ dotnet build
MSBuild version 17.7.0+5785ed5c2 for .NET
  Determining projects to restore...
  Restored /home/rich/git/dotnet-docker/samples/aspnetapp/aspnetapp/aspnetapp.csproj (in 595 ms).
/usr/share/dotnet/sdk/8.0.100-preview.6.23330.14/Sdks/Microsoft.NET.Sdk/targets/Microsoft.NET.RuntimeIdentifierInference.targets(314,5): message NETSDK1057: You are using a preview version of .NET. See: https://aka.ms/dotnet-support-policy [/home/rich/git/dotnet-docker/samples/aspnetapp/aspnetapp/aspnetapp.csproj]
  aspnetapp -> /home/rich/git/dotnet-docker/samples/aspnetapp/aspnetapp/bin/Debug/net8.0/aspnetapp.dll

Build succeeded.
    0 Warning(s)
    0 Error(s)

Time Elapsed 00:00:04.36

jaredpar commented 1 year ago

The other issue with the preview message is that it displays once for every project being built, not once per build. That can really clutter up your build output when building large projects.

richlander commented 1 year ago

100% true. That part is super annoying.

https://github.com/dotnet/sdk/issues/10352

dotnet / sdk

Report "preview" more classy #34526