Closed Forgind closed 1 year ago
@Forgind, do you plan to finalize this PR?
@Forgind, do you plan to finalize this PR?
I think this was a worthwhile project in getting people to not make this (rather confusing) mistake, but I didn't know how to actually ship the analyzer such that it would run on users' code. I think it might be worthwhile to ship this (and I can take Youssef1313's comments into account if so) if someone can offer a suggestion as to how to make the analyzer run for users?
I think it is still a reasonable idea to have an analyzer that helps people use MSBuildLocator. This PR may be a good starting point for that, or it may be stale enough that it's easier to start over. Either way, I don't think we're planning to invest in an analyzer right now and I'd be ok closing this until we are.
Close with the comment: https://github.com/microsoft/MSBuildLocator/pull/116#issuecomment-1593615775
Still to do: drop into vsix.
Fixes #67