chore: fix some lints

Comparing with > and max seem to have the same failure mode around one of the comparands being None, so the PLR1730 lint fix seems safe. Signed-off-by: Will Murphy <[email protected]>
anchore · Oct 3, 2024 · 16dc74a · 16dc74a
1 parent 927265f
commit 16dc74a
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 2 deletions.
diff --git a/example/awesome/parser.py b/example/awesome/parser.py
@@ -5,6 +5,7 @@
 
 import orjson
 import requests
+
 from vunnel import utils, workspace
 
 # NOTE, CHANGE ME!: this namespace should be unique to your provider and match expectations from

diff --git a/example/run.py b/example/run.py
@@ -5,6 +5,7 @@
 
 import awesome
 import orjson
+
 from vunnel import provider, result
 
 fakedata = [

diff --git a/src/vunnel/providers/ubuntu/parser.py b/src/vunnel/providers/ubuntu/parser.py
@@ -562,8 +562,7 @@ def map_parsed(parsed_cve: CVEFile, logger: logging.Logger | None = None):  # no
             # Check for max priority of all packages with it set
             if p.priority:
                 pkg_sev = getattr(Severity, p.priority.capitalize())
-                if pkg_sev > r.Severity:
-                    r.Severity = pkg_sev
+                r.Severity = max(pkg_sev, r.Severity)
 
     return set(vulns.values())