dotnet/try-convert

Bring more prominence to messages that can help developers cost/plan for significant code changes

cathysull opened this issue · 1 comments

cathysull commented

Thoughts on calling out places in the experience where we say "significant work will be required" in red?

Example:
[14:06:19 INF] Applying upgrade step Convert project file to SDK style
[14:06:19 INF] Converting project file format with try-convert
[14:06:20 INF] [try-convert] C:\Users\me\Source\Repos\eShopModernizing\eShopLegacyMVCSolution\src\eShopLegacyMVC\eShopLegacyMVC.csproj contains a reference to System.Web, which is not supported on .NET Core. You may have significant work ahead of you to fully port this project.

Ideas to bring more prominence to these messages:

  • Red font color
  • Log them as warnings (would give them a yellow prefix)
  • Pull them out of try-convert and surface them up in the Upgrade Assistant instead (with visual treatment)
cartermp commented

I think it should be pretty straightforward to make the message a little more prominent. A ConsoleColor.Yellow call would probably suffice.