Skip to content

DB Version Check Helper (GSI-2456)#243

Merged
TheByronHimes merged 6 commits into
mainfrom
feature/db_version_check_GSI-2456
Jul 10, 2026
Merged

DB Version Check Helper (GSI-2456)#243
TheByronHimes merged 6 commits into
mainfrom
feature/db_version_check_GSI-2456

Conversation

@TheByronHimes

@TheByronHimes TheByronHimes commented Jul 10, 2026

Copy link
Copy Markdown
Member

This introduces a check_db_version() utility for the migration tools. Its purpose is to provide a convenient way to enforce DB state in application code independent of the checks that occur in migrations themselves.

@TheByronHimes
TheByronHimes force-pushed the feature/db_version_check_GSI-2456 branch from 47a5017 to b093b02 Compare July 10, 2026 14:07
@coveralls

coveralls commented Jul 10, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 29102201331

Coverage increased (+0.02%) to 93.325%

Details

  • Coverage increased (+0.02%) from the base build.
  • Patch coverage: 38 of 38 lines across 3 files are fully covered (100%).
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 3970
Covered Lines: 3705
Line Coverage: 93.32%
Coverage Strength: 0.93 hits per line

💛 - Coveralls

@TheByronHimes
TheByronHimes requested a review from Cito July 10, 2026 14:20
@TheByronHimes
TheByronHimes marked this pull request as ready for review July 10, 2026 14:20
Cito
Cito previously approved these changes Jul 10, 2026

@Cito Cito left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Just one minor issue, see below.

Comment thread src/hexkit/providers/mongodb/migrations/_utils.py Outdated
@TheByronHimes
TheByronHimes merged commit 4043288 into main Jul 10, 2026
8 checks passed
@TheByronHimes
TheByronHimes deleted the feature/db_version_check_GSI-2456 branch July 10, 2026 15:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants