Skip to content

Remove AbstractMigrationBuilder + Mark readonly properties + Cleanup#311

Merged
vjik merged 10 commits intomasterfrom
cleanup
Dec 9, 2025
Merged

Remove AbstractMigrationBuilder + Mark readonly properties + Cleanup#311
vjik merged 10 commits intomasterfrom
cleanup

Conversation

@vjik
Copy link
Member

@vjik vjik commented Dec 9, 2025

Q A
Is bugfix?
New feature?
Breaks BC?

@codecov
Copy link

codecov bot commented Dec 9, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.69%. Comparing base (0c5303a) to head (815536a).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files
@@             Coverage Diff              @@
##             master     #311      +/-   ##
============================================
+ Coverage     95.14%   99.69%   +4.55%     
+ Complexity      323      299      -24     
============================================
  Files            21       20       -1     
  Lines          1029      985      -44     
============================================
+ Hits            979      982       +3     
+ Misses           50        3      -47     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@vjik vjik requested a review from a team December 9, 2025 06:26
@vjik vjik added the status:code review The pull request needs review. label Dec 9, 2025
@vjik vjik changed the title Remove AbstractMigrationBuilder constructor + Mark readonly properties + Cleanup Remove AbstractMigrationBuilder + Mark readonly properties + Cleanup Dec 9, 2025
@vjik vjik requested a review from Tigrov December 9, 2025 08:23
@vjik vjik merged commit e27309b into master Dec 9, 2025
50 of 51 checks passed
@vjik vjik deleted the cleanup branch December 9, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

status:code review The pull request needs review.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants