Closed Piedone closed 1 year ago
Some notes:
<GlobalAnalyzerConfigFiles>
and <EditorConfigFiles>
seem to be virtually equivalent. Even if you use the latter and name your files something.editorconfig
, they'll actually be global AnalyzerConfig files and need is_global = true
. GlobalAnalyzerConfigFiles
is meant to be used by us.<GlobalAnalyzerConfigFiles>
.We'll have a hard time getting the editor GUI working: https://github.com/dotnet/roslyn/issues/70326
We need .NET 8 for this: https://github.com/dotnet/roslyn-analyzers/issues/6281#issuecomment-1381351146
We don't need .NET 8, that's only when TreatWarningsAsErrors=false and CodeAnalysisTreatWarningsAsErrors=true but we don't do that.
This is only suitable once .editorconfig has everything ruleset files provide already:
<EditorConfigFiles>
could help us, see this blog post. An alternative would be https://github.com/dotnet/roslyn/issues/19028#issuecomment-843571161..globalconfig
, but that's only for analyzer configs, not edit configuration like charset or guidelines. See https://github.com/dotnet/roslyn/issues/47707.Tasks:
<developmentDependency>true</developmentDependency>
in the nuspec template. The analyzer packages we use do this. Check the generated<PackageReference>
after a new install and change the docs, including release notes, and consuming projects if necessary.