Skip to content

Conversation

@faizanazim11
Copy link

This pull request updates dependencies and enhances the database session factory methods to support a new raw_db option. This option allows bypassing the tenant ID when creating database sessions, which can be useful for system-level operations or migrations. The changes also include a version bump for the package.

Dependency updates:

  • .pre-commit-config.yaml: Upgraded pre-commit-hooks from v5.0.0 to v6.0.0 and ruff-pre-commit from v0.11.0 to v0.14.4 for improved linting and formatting support.

Database session management improvements:

Version update:

@faizanazim11 faizanazim11 self-assigned this Nov 12, 2025
Copilot AI review requested due to automatic review settings November 12, 2025 16:15
@faizanazim11 faizanazim11 added the bug Something isn't working label Nov 12, 2025
Copilot finished reviewing on behalf of faizanazim11 November 12, 2025 16:18
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR introduces a raw_db parameter to the get_db_factory methods in both sync and async session managers, allowing database sessions to bypass tenant ID scoping. The PR also includes dependency updates for pre-commit hooks and a version bump.

  • Addition of raw_db boolean parameter to enable system-level database operations without tenant isolation
  • Dependency updates: pre-commit-hooks (v5.0.0 → v6.0.0) and ruff-pre-commit (v0.11.0 → v0.14.4)
  • Version bump from 1.3.1 to 1.3.2

Reviewed Changes

Copilot reviewed 4 out of 6 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
sql_db_utils/session_management.py Added raw_db parameter to get_db_factory method to bypass tenant_id when creating sessions
sql_db_utils/asyncio/session_management.py Added raw_db parameter to async get_db_factory method, maintaining sync/async parity
sql_db_utils/__version__.py Bumped version from 1.3.1 to 1.3.2
.pre-commit-config.yaml Updated pre-commit-hooks to v6.0.0 and ruff-pre-commit to v0.14.4
.github/copilot-instructions.md Removed trailing whitespace from several lines
uv.lock New lock file added with complete dependency tree for the project

faizanazim11 and others added 2 commits November 12, 2025 21:49
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@faizanazim11 faizanazim11 merged commit 10fa85f into main Nov 12, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants