Closed Forgind closed 1 year ago
I could expose instance as a parameter as I have ContainsRoslynCompiler exposed here. Would you want the non-VS versions to return null for that?
Hi @rainersigwald and @Forgind,
Do we plan to work on this PR? Can we close it?
I think we need to have a design for #22 before taking any implementation and would favor closing this.
I think this is low enough priority that it doesn't warrant a serious design discussion. I'd personally prefer to just take this roughly as-is over closing it, but closing it makes more sense than talking about it for too long.
Fixes #22
I didn't see an obvious way to determine whether the C# compiler was installed for the dotnet sdk, and I didn't want to replicate all the logic from VSLocationHelper for a command prompt. Maybe right?