Project-system: Non-informative warning icons in solution explorer

Created on 22 May 2017  路  9Comments  路  Source: dotnet/project-system

Something went wrong with a CPS-to-CPS p2p reference in my IDE:

image

However, there is no information that can be ascertained from these warning icons. The only warnings i have are:

image

Bug Feature-Dependency-Node

Most helpful comment

We should have a rule; if there's a warning/error symbol in the Dependency node - there must to an actionable error/warning in the Error List. I'm seeing warnings everywhere in our tree - and no idea what they mean.

All 9 comments

@natidea mentioned that he saw a bug where the project reference was showing the error but the project wasn't. The analyzer warning issue is https://github.com/dotnet/project-system/issues/1634 and I think it's because of non-normalized paths being treated incorrectly by the dependencies node.

We should have a rule; if there's a warning/error symbol in the Dependency node - there must to an actionable error/warning in the Error List. I'm seeing warnings everywhere in our tree - and no idea what they mean.

Agreed, but post-15.6 at this point.

Here's another example:

Repro:

  1. Clone and sync to https://github.com/rosieks/JavaScriptServices/commit/087183ea18ed563d03d35266af081a5adb24eca1
  2. Open JavaScriptServices.sln
  3. Expand WebPack

image

It looks like the failed resolution of the "SDK" is affecting the NuGet node.

@davkean You found a separate issue actually: since sdk and packages are related, you are right that warning on Nuget group could be triggered by sdk. This is different from the original complain in current issue, when there no actual errors/warnings when any node has warning sign. I would open a separate issue to track your scenario..

I've filed https://github.com/dotnet/project-system/issues/2398 to track that last example.

It's especially confusing when the warning icon is on the Analyzers node, because that led me to believe the nodes under it represented warnings found in my code.

This is quite a general issue, and a lot of stuff around this has been fixed recently. Without a repro to test against, I can't tell whether this is still happening.

We have some other issues to track the general case covered here. I suspect #5044 is the best fit here. Regarding the lack of information about the error itself we have #4357 and #3780. I'll close this. If someone has a repro that isn't covered by an existing issue, please file a new one.

Was this page helpful?
0 / 5 - 0 ratings