Mention globalconfig in the docs, and tweak the detection for consistency (#224)

* Add globalconfig for Rider (it doesn't seem to work otherwise)

* Make bool check consistent with other checks

* Tweak README

Author: andrewlock

README.md

@@ -283,17 +283,14 @@ You can override the name of the extension class by setting `ExtensionClassName`
 
 _NetEscapades.EnumGenerators_ includes optional analyzers that encourage the use of the generated extension methods instead of the built-in `System.Enum` methods. These analyzers can help improve performance by suggesting the faster, generated, alternatives like `ToStringFast()`, `HasFlagFast()`, and `TryParse()`.
 
-### Enabling the analyzers
+Alternatively, [add a `.globalconfig` file](https://learn.microsoft.com/dotnet/fundamentals/code-analysis/configuration-files#global-analyzerconfig) to your project with the following content:
 
-The usage analyzers are disabled by default. To enable them, set the `EnumGenerator_EnableUsageAnalyzers` MSBuild property to `true` in your project:
-
-```xml
-<PropertyGroup>
-  <EnumGenerator_EnableUsageAnalyzers>true</EnumGenerator_EnableUsageAnalyzers>
-</PropertyGroup>
+```ini
+is_global = true
+build_property.EnumGenerator_EnableUsageAnalyzers = true
 ```
 
-After adding this configuration, the analyzers in your project should be enabled with the default severity of `Warning`.
+After using one of these configuration options, the analyzers in your project should be enabled with the default severity of `Warning`.
 
 ### Configuring analyzer severity (optional)
 

src/NetEscapades.EnumGenerators/Diagnostics/UsageAnalyzers/UsageAnalyzerConfig.cs

@@ -9,7 +9,6 @@ public static class UsageAnalyzerConfig
     public const string EnableKey = "build_property.EnumGenerator_EnableUsageAnalyzers";
 
     internal static bool IsEnabled(AnalyzerOptions context)
-        => context.AnalyzerConfigOptionsProvider.GlobalOptions.TryGetValue(EnableKey, out var value) &&
-           bool.TryParse(value, out var isEnabled)
-           && isEnabled;
+        => context.AnalyzerConfigOptionsProvider.GlobalOptions.TryGetValue(EnableKey, out var value)
+           && string.Equals(value, "true", StringComparison.OrdinalIgnoreCase);
 }

src/NetEscapades.EnumGenerators/README.md

@@ -314,7 +314,14 @@ The usage analyzers are disabled by default. To enable them, set the `EnumGenera
 </PropertyGroup>
 ```
 
-After adding this configuration, the analyzers in your project should be enabled with the default severity of `Warning`.
+Alternatively, [add a `.globalconfig` file](https://learn.microsoft.com/dotnet/fundamentals/code-analysis/configuration-files#global-analyzerconfig) to your project with the following content:
+
+```ini
+is_global = true
+build_property.EnumGenerator_EnableUsageAnalyzers = true
+```
+
+After using one of these configuration options, the analyzers in your project should be enabled with the default severity of `Warning`.
 
 ### Configuring analyzer severity (optional)
 

tests/NetEscapades.EnumGenerators.IntegrationTests/.globalconfig

@@ -0,0 +1,2 @@
+is_global = true
+build_property.EnumGenerator_EnableUsageAnalyzers = true