docs: Adds comments re LiteralString

#3526 (review)
vega · Sep 5, 2024 · 9ce798c · 9ce798c
1 parent 027c82c
commit 9ce798c
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 0 deletions.
diff --git a/altair/utils/theme.py b/altair/utils/theme.py
@@ -19,6 +19,8 @@
 ThemeType = Plugin[Dict[str, Any]]
 
 
+# HACK: See for `LiteralString` requirement in `name`
+# https://github.com/vega/altair/pull/3526#discussion_r1743350127
 class ThemeRegistry(PluginRegistry[ThemeType, Dict[str, Any]]):
     def enable(
         self, name: LiteralString | AltairThemes | VegaThemes | None = None, **options

diff --git a/altair/vegalite/v5/theme.py b/altair/vegalite/v5/theme.py
@@ -74,6 +74,8 @@ def __repr__(self) -> str:
 themes.enable("default")
 
 
+# HACK: See for `LiteralString` requirement in `name`
+# https://github.com/vega/altair/pull/3526#discussion_r1743350127
 def register_theme(
     name: LiteralString, *, enable: bool
 ) -> Callable[[Callable[P, R]], Callable[P, R]]: