dbapi/Cursor: fix execute default args value by odeke-em · Pull Request #34 · googleapis/python-spanner-django
Navigation Menu
This repository was archived by the owner on Mar 20, 2026. It is now read-only.
- Notifications You must be signed in to change notification settings
- Fork 32
Conversation
Reported by @timgraham while working on the Django adapter, the prior signature of: def execute(self, *args, **kwargs) was passing args=() into self.__do_execute_non_update(sql, *args, **kwargs) so changed it to def execute(self, args=None)
odeke-em
deleted the
fix-Cursor.execute-signature
branch
This was referenced
Mar 31, 2026This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters