Josef Pihrt

Results 213 comments of Josef Pihrt

https://open-vsx.org/extension/josefpihrt-vscode/roslynator

Thanks for answer. # Analyzers I agree that analyzers should be consumed through nuget packages. # Refactorings I didn't know that refactorings could be distributed as a part of nuget...

The question is not whether it should be possible to distribute refactorings as a part of nuget. The question is whether it should be possible to distribute them as a...

Why to make these workarounds when it could be supported by VS Code just as it's supported by Visual Studio? What is the argument to prefer distributing common refactorings/fixes as...

> @josefpihrt You could probably create an MSBuild targets file in one of the extension locations that adds a reference to the refactorings DLL to all projects that get loaded....

Roslynator analyzers now supports Roslyn 3.8 (see #1349)

> Confirmed here also, and I think that: > > ```cs > arguments.Append($" CONTENTDIR=\"{destinationPath}\""); > ``` > > Really does look much clearer than: > > ```cs > arguments.Append(" CONTENTDIR=\"").Append(destinationPath).Append('\"');...

@dhabierre Please check format of your roslynatorconfig file: https://github.com/JosefPihrt/Roslynator/blob/ab443617938c40b1ac87a612da054cd7837a82c9/docs/Configuration.md?plain=1#L41-L42 Following format is correct: ``` is_global = true roslynator_analyzers.enabled_by_default = true ```

@fgimian Default configuration file (`.roslynatorconfig`) works only with VS extension (or VS Code extension): https://github.com/JosefPihrt/Roslynator/blob/ab443617938c40b1ac87a612da054cd7837a82c9/docs/Configuration.md?plain=1#L37 Do you use VS extension? If you use nuget packages it won't work. Regarding the...

I think that these diagnostic rules cannot be used both at the same time.