Skip to content

Refine cursor function typing#50

Open
sutyum wants to merge 1 commit intomainfrom
codex/update-type-definitions-and-run-mypy
Open

Refine cursor function typing#50
sutyum wants to merge 1 commit intomainfrom
codex/update-type-definitions-and-run-mypy

Conversation

@sutyum
Copy link
Member

@sutyum sutyum commented May 17, 2025

Summary

  • narrow CursorExecFunction to use sqlite3.Cursor and sqlite3.Connection
  • import this alias in SQLite database implementation
  • update accompanying type stub

Testing

  • ruff format personal_graph/database/db.py personal_graph/database/sqlite/sqlite.py stub/personal_graph/database/sqlite/sqlite.pyi
  • ruff check personal_graph/database/db.py personal_graph/database/sqlite/sqlite.py stub/personal_graph/database/sqlite/sqlite.pyi
  • poetry run python -m mypy . (fails: No module named mypy)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant