search

CBielstein / APRSsharp

APRS# - Modern APRS software for the amateur radio community
MIT License
12 stars 5 forks source link

Enable additional static analysis analyzers #27

Open CBielstein opened 4 years ago

CBielstein commented 4 years ago
CBielstein commented 4 years ago

As pointed out in a PR comment here: https://github.com/CBielstein/APRSsharp/pull/19#discussion_r445875891

CBielstein commented 4 years ago

Maybe investigate others here: https://github.com/dotnet/roslyn-analyzers. I know I've also seen things like IDE0044 being highlighted in vscode, it'd be great to surface some of those in the build.

CBielstein commented 3 years ago

As per conversation here: https://github.com/dotnet/roslyn/issues/33558 and here https://docs.microsoft.com/en-us/dotnet/fundamentals/code-analysis/overview#enable-on-build, enabling more analyzers on the build (such as the IDExxxx error codes) requires .NET 5 or higher. This is similar to #81, which would also require a newer .NET version.