Address feedback

starlark.c

@@ -225,7 +225,10 @@ PyDoc_STRVAR(
     "* Python :py:obj:`python:set` to Starlark `set "
     "<https://pkg.go.dev/go.starlark.net/starlark#Set>`_\n"
     "* Python :py:obj:`python:tuple` to Starlark `tuple "
-    "<https://pkg.go.dev/go.starlark.net/starlark#Tuple>`_\n\n"
+    "<https://pkg.go.dev/go.starlark.net/starlark#Tuple>`_\n"
+    "* Python functions can be registered as a Starlark function directly. "
+    "Any exceptions raised will be rethrown as :py:class:`EvalError`.\n"
+    "\n"
     "For the aggregate types (``dict``, ``list``, ``set``, and ``tuple``,) all keys "
     "and/or values must also be one of the supported types.\n\n"
     "Attempting to set a value of any other Python type will raise a "

tests/test_set_globals.py

@@ -1,3 +1,5 @@
+import sys
+
 import pytest
 
 from starlark_go import EvalError, Starlark, StarlarkError
@@ -188,10 +190,12 @@ def func_impl(x):
     with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: got zero"):
         s.eval("func(0)")
 
-    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: test_func.<locals>.func_impl\(\) missing 1 required positional argument: 'x'"):
+    name = "func_impl" if sys.version_info < (3, 10) else "test_func.<locals>.func_impl"
+
+    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: " + name + r"\(\) missing 1 required positional argument: 'x'"):
         s.eval("func()")
 
-    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: test_func.<locals>.func_impl\(\) got an unexpected keyword argument 'unknown'"):
+    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: " + name + r"\(\) got an unexpected keyword argument 'unknown'"):
         s.eval("func(unknown=0)")
 
 
@@ -240,10 +244,12 @@ def func_impl(self, x):
     with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: got zero"):
         s.exec("func(0)")
 
-    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: test_method.<locals>.Test.func_impl\(\) missing 1 required positional argument: 'x'"):
+    name = "func_impl" if sys.version_info < (3, 10) else "test_method.<locals>.Test.func_impl"
+
+    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: " + name + r"\(\) missing 1 required positional argument: 'x'"):
         s.eval("func()")
 
-    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: test_method.<locals>.Test.func_impl\(\) got an unexpected keyword argument 'unknown'"):
+    with pytest.raises(EvalError, match=r"<builtin> in func_impl:0:0: " + name + r"\(\) got an unexpected keyword argument 'unknown'"):
         s.eval("func(unknown=0)")