From de8b86de4af2aa1e71676b30fae173826d60943c Mon Sep 17 00:00:00 2001
From: Eli Benevedes <ebenevedes@blueorigin.com>
Date: Fri, 18 Nov 2022 16:58:22 -0800
Subject: [PATCH] Allow for missing ruleId. This is allowed in the sarif spec,
 and may be introduced in ament_clang_tidy.

---
 process_sarif/sarif.py | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/process_sarif/sarif.py b/process_sarif/sarif.py
index 4067fb6..fd64e1d 100644
--- a/process_sarif/sarif.py
+++ b/process_sarif/sarif.py
@@ -302,7 +302,7 @@ def to_dict(self) -> dict:
 
     @staticmethod
     def from_dict(result_dict: dict, artifacts: List[Artifact], tool: Tool, verbose=True, log_path: Optional[str] = None) -> "Result":
-        ruleId = result_dict["ruleId"]
+        ruleId = result_dict["ruleId"] if "ruleId" in result_dict else ""
         level = Level.from_str(result_dict["level"]) if "level" in result_dict else Level.UNKNOWN
         kind = ResultKind.from_str(result_dict["kind"]) if "kind" in result_dict else ResultKind.UNKNOWN
         message = result_dict["message"]["text"] if "text" in result_dict["message"] else ""