Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

MSB4011 could be shown nested in the import tree #774

Open
rainersigwald opened this issue Apr 17, 2024 · 1 comment
Open

MSB4011 could be shown nested in the import tree #774

rainersigwald opened this issue Apr 17, 2024 · 1 comment

Comments

@rainersigwald
Copy link
Contributor

rainersigwald commented Apr 17, 2024

MSB4011.zip renders like this:

image

I think it would be helpful to have that warning under the associated file node, that is nest it under the "location" value of the warning rather than (or maybe in addition to?) the "evaluation warnings for this project" node.

Text log rendering shows this structure slightly better than the viewer today since it interleaves the warnings with the imports, showing their context.

@KirillOsenkov
Copy link
Owner

Sounds reasonable.

One could argue that it might be useful to have all the warnings bunched up together, but you could achieve that with $warning under(id:35) anyway (or "Search under node")

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants