Skip to content

feat: warn on destructive and unsupported hypertable migration changes - #68

Merged
sebastian-ederer merged 1 commit into
mainfrom
feat/improve_warnings_for_destructive_operations
Oct 7, 2026
Merged

sebastian-ederer merged 1 commit into
mainfrom
feat/improve_warnings_for_destructive_operations

Conversation

@sebastian-ederer

Copy link
Copy Markdown
Member

Description

Surface migration changes that destroy data or cannot be applied, instead
of silently skipping or failing them.

Type of Change

  • Bug fix (non-breaking change addressing an issue)
  • New feature (non-breaking change adding functionality)
  • Refactoring
  • Documentation
  • CI / build / tooling
  • Breaking change (fix or feature causing existing functionality to change)

Test Plan

  • Unit tests added/updated
  • Integration tests added/updated (Testcontainers)
  • Existing tests pass (dotnet test)

Checklist

  • Code follows the project's coding styles and guidelines
  • I have updated the documentation accordingly
  • I have added tests to cover my changes
  • No new compiler warnings introduced
  • Public API changes have XML documentation

@codecov

codecov Bot commented Oct 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 98.72611% with 2 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
.../Eftdb/Internals/TimescaleMigrationsModelDiffer.cs 0.00% 1 Missing ⚠️
src/Eftdb/TimescaleDbMigrationsSqlGenerator.cs 95.83% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@sebastian-ederer
sebastian-ederer merged commit a29ee14 into main Oct 7, 2026
3 checks passed
@sebastian-ederer
sebastian-ederer deleted the feat/improve_warnings_for_destructive_operations branch October 7, 2026 21:50
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.

1 participant