Skip to content

Conversation

@cofin
Copy link
Member

@cofin cofin commented Jan 23, 2026

  • Added summary_only parameter to BaseMigrationRunner to control logging verbosity.
  • Introduced _log_migration_event method to centralize logging logic and suppress info logs in summary-only mode.
  • Updated SyncMigrationRunner and AsyncMigrationRunner to utilize the new logging method for migration events.
  • Modified SyncMigrationTracker and AsyncMigrationTracker to conditionally echo migration schema changes based on _should_echo.
  • Enhanced validate_migration_order function to support logging via a logger and conditional console output.

cofin added 3 commits January 23, 2026 00:06
- Added `summary_only` parameter to `BaseMigrationRunner` to control logging verbosity.
- Introduced `_log_migration_event` method to centralize logging logic and suppress info logs in summary-only mode.
- Updated `SyncMigrationRunner` and `AsyncMigrationRunner` to utilize the new logging method for migration events.
- Modified `SyncMigrationTracker` and `AsyncMigrationTracker` to conditionally echo migration schema changes based on `_should_echo`.
- Enhanced `validate_migration_order` function to support logging via a logger and conditional console output.
@cofin cofin merged commit 742ca34 into main Jan 23, 2026
25 of 26 checks passed
@cofin cofin deleted the fix/printing branch January 23, 2026 17:42
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.

2 participants