Copilot commented on code in PR #2374:
URL: https://github.com/apache/age/pull/2374#discussion_r3066155520
##########
drivers/python/age/age.py:
##########
@@ -162,6 +162,71 @@ def setUpAge(conn:psycopg.connection, graphName:str,
load_from_plugins:bool=Fals
if graphName != None:
checkGraphCreated(conn, graphName)
+
+def configure_connection(
+ conn: psycopg.connection,
+ graph_name: str | None = None,
+ load: bool = False,
+ load_from_plugins: bool = False,
Review Comment:
`configure_connection()` uses PEP 604 union syntax (`str | None`) in its
public signature. This is a syntax error on Python 3.9, but the package
metadata declares `requires-python = ">=3.9"`
(drivers/python/pyproject.toml:25). Either adjust the type hints to be
3.9-compatible (e.g., `Optional[str]` / `Union[...]`) or bump the declared
minimum Python version to 3.10+ so users on 3.9 don’t break at import time.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
