Use ternary

AndreasArvidsson · AndreasArvidsson · commit 562178575c65 · 2025-02-06T05:18:21.000+01:00
diff --git a/cursorless-talon/src/snippets/snippet_types.py b/cursorless-talon/src/snippets/snippet_types.py
@@ -10,12 +10,10 @@ class ScopeType:
     type: str
 
 
-def to_scope_types(scope_types: str | list[str] | None) -> list[ScopeType] | None:
+def to_scope_types(scope_types: str | list[str]) -> list[ScopeType]:
     if isinstance(scope_types, str):
         return [ScopeType(scope_types)]
-    elif scope_types is not None:
-        return [ScopeType(st) for st in scope_types]
-    return None
+    return [ScopeType(st) for st in scope_types]
 
 
 # Community types
@@ -54,7 +52,7 @@ def create(
     ):
         return CustomInsertionSnippet(
             snippet.body,
-            to_scope_types(snippet.scopes),
+            to_scope_types(snippet.scopes) if snippet.scopes else None,
             # languages will be missing if the user has an older version of community
             snippet.languages if hasattr(snippet, "languages") else None,
             substitutions=substitutions,
diff --git a/cursorless-talon/src/snippets/snippets.py b/cursorless-talon/src/snippets/snippets.py
@@ -125,7 +125,7 @@ def cursorless_insert_snippet(
         """Cursorless: Insert custom snippet <body>"""
         snippet = CustomInsertionSnippet(
             body,
-            to_scope_types(scope_type),
+            to_scope_types(scope_type) if scope_type else None,
             languages=None,
             substitutions=None,
         )

Original file line number	Diff line number	Diff line change
`@@ -125,7 +125,7 @@ def cursorless_insert_snippet(`
`125`	`125`	`"""Cursorless: Insert custom snippet <body>"""`
`126`	`126`	`snippet = CustomInsertionSnippet(`
`127`	`127`	`body,`
`128`		`- to_scope_types(scope_type),`
	`128`	`+ to_scope_types(scope_type) if scope_type else None,`
`129`	`129`	`languages=None,`
`130`	`130`	`substitutions=None,`
`131`	`131`	`)`