Allow '_' and '-' in named placeholders. (#71)

Fixes 69

CHANGELOG.md

@@ -6,6 +6,9 @@
 
 * Removed the PartialAppAnalyzer. [#68](https://github.com/G-Research/fsharp-analyzers/issues/68)
 
+### Fixed
+* Fixed a false positive of LoggingTemplateMissingValuesAnalyzer. [#69](https://github.com/G-Research/fsharp-analyzers/issues/69)
+
 ## 0.8.0 - 2024-01-30
 
 ### Fixed

src/FSharp.Analyzers/LoggingTemplateMissingValuesAnalyzer.fs

@@ -34,7 +34,7 @@ let analyze (typedTree : FSharpImplementationFileContents) =
                 "Microsoft.Extensions.Logging.LoggerExtensions.LogWarning"
             ]
 
-    let pattern = @"(?<opening>{+)[a-zA-Z0-9]*(?<closing>}+)"
+    let pattern = @"(?<opening>{+)[a-zA-Z0-9_-]*(?<closing>}+)"
     let regex = Regex pattern
 
     let walker =

tests/FSharp.Analyzers.Tests/data/loggingtemplatemissingvalues/negative/No warnings for unusual template names.fs

@@ -0,0 +1,9 @@
+module M
+
+    open Microsoft.Extensions.Logging
+
+    let testlog () =
+        use factory = LoggerFactory.Create(fun b -> b.AddConsole() |> ignore)
+        let logger: ILogger = factory.CreateLogger("Program")
+
+        logger.Log(LogLevel.Information, "xxx {o-ne} yyy {t_wo}", 23, 42)