Merge pull request #700 from prospector-dev/blender-line-none

sbrunner · web-flow · commit bd3a18ce9a3a · 2024-11-14T10:07:20.000+01:00
Fix blender when line is None
diff --git a/prospector/blender.py b/prospector/blender.py
@@ -88,8 +88,8 @@ def blend(messages: list[Message], blend_combos: Optional[list[list[tuple[str, s
     msgs_grouped: dict[Path, dict[int, list[Message]]] = defaultdict(lambda: defaultdict(list))
 
     for message in messages:
-        assert message.location.line is not None
-        msgs_grouped[message.location.path][message.location.line].append(
+        line = message.location.line
+        msgs_grouped[message.location.path][-1 if line is None else line].append(
             message,
         )
 

Original file line number	Diff line number	Diff line change
`@@ -88,8 +88,8 @@ def blend(messages: list[Message], blend_combos: Optional[list[list[tuple[str, s`
`88`	`88`	`msgs_grouped: dict[Path, dict[int, list[Message]]] = defaultdict(lambda: defaultdict(list))`
`89`	`89`
`90`	`90`	`for message in messages:`
`91`		`- assert message.location.line is not None`
`92`		`- msgs_grouped[message.location.path][message.location.line].append(`
	`91`	`+ line = message.location.line`
	`92`	`+ msgs_grouped[message.location.path][-1 if line is None else line].append(`
`93`	`93`	`message,`
`94`	`94`	`)`
`95`	`95`