Analyzer doesn't change the syntax tree, so we should use the Visitor #10
Conversation
…, not the Rewriter
|
@eNeRGy164 Thank you for your contribution. These changes look ok to me. Unfortunately it seems the build is failing, although I'm not entirely sure why. My best guess is that we'll need to update some things to make this work properly because of changes made to .NET/Visual Studio. At the moment we build a .NET Framework based version for Windows and a .NET Core based version for the other platforms. This was due to the absence of MSBuildWorkspace in .NET Core. I believe that with Roslyn 3.0 this is no longer an issue, so perhaps we should merge these, but I haven't had a change to work on that yet. I don't have much time to look at this right now unfortunately. |
|
Looks this StackOverflow comment might point us at a solution. Maybe we can switch to using dotnet restore rather than NuGet restore in the pipeline (azure-pipeline.yml)? |
Related to #9