Merge pull request #50 from dawedawe/option_name_exn

deal with options and properties that might throw

Author: dawedawe

CHANGELOG.md

@@ -4,6 +4,7 @@
 
 ### Fixed
 * Handle types without a FullName more gracefully in the EmptyStringAnalyzer. [#48](https://github.com/ionide/ionide-analyzers/pull/48)
+* Handle types without a FullName more gracefully in the HandleOptionGracefullyAnalyzer. [#50](https://github.com/ionide/ionide-analyzers/pull/50)
 
 ## 0.5.0 - 2023-11-23
 

src/Ionide.Analyzers/Suggestion/HandleOptionGracefullyAnalyzer.fs

@@ -17,7 +17,13 @@ let optionGetAnalyzer (ctx: CliContext) =
             { new TypedTreeCollectorBase() with
                 override x.WalkCall _ (mfv: FSharpMemberOrFunctionOrValue) _ _ (args: FSharpExpr list) (m: range) =
                     let fullyQualifiedCall =
-                        String.Join(".", mfv.DeclaringEntity.Value.FullName, mfv.DisplayName)
+                        let fullName =
+                            mfv.DeclaringEntity
+                            |> Option.map (fun e -> e.TryFullName)
+                            |> Option.flatten
+                            |> Option.defaultValue ""
+
+                        String.Join(".", fullName, mfv.DisplayName)
 
                     if
                         (mfv.FullName = "Microsoft.FSharp.Core.Option.get"