chore: add literal_eval check

Signed-off-by: Henry Schreiner <[email protected]>
wojdyr · Aug 22, 2023 · d836529 · d836529
1 parent 4903898
commit d836529
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/docs/ext/conftabs.py b/docs/ext/conftabs.py
@@ -1,5 +1,6 @@
 from __future__ import annotations
 
+import ast
 import textwrap
 
 from docutils import nodes
@@ -14,7 +15,7 @@ class ConfTabs(SphinxDirective):
     def run(self):
         name, result = self.arguments
         env_name = f"SKBUILD_{name.replace('.', '_').upper()}"
-        value_result = eval(result, {}, {})  # noqa: PGH001
+        value_result = ast.literal_eval(result)
         if isinstance(value_result, list):
             joined_result = ";".join(value_result)
         elif isinstance(value_result, bool):

diff --git a/pyproject.toml b/pyproject.toml
@@ -223,6 +223,7 @@ select = [
     "UP",          # pyupgrade
     "YTT",         # flake8-2020
     "ANN204",      # Add -> None to __init__
+    "S307",        # eval -> literal_eval
 ]
 extend-ignore = [
     "PLR",     # Design rules for pylint