Skip to content

fix: drop constraint for delete count migration#1185

Merged
ferhatelmas merged 1 commit into
masterfrom
ferhat/lock-delete-count
Jun 29, 2026
Merged

fix: drop constraint for delete count migration#1185
ferhatelmas merged 1 commit into
masterfrom
ferhat/lock-delete-count

Conversation

@ferhatelmas

Copy link
Copy Markdown
Member

What kind of change does this PR introduce?

Bug fix

What is the current behavior?

Constraint will lock the table for too long.

What is the new behavior?

Drop the constraint and application does the guarding.

Additional context

Simplify tests by using common payload.

Copilot AI review requested due to automatic review settings June 29, 2026 21:57
@ferhatelmas ferhatelmas requested a review from a team as a code owner June 29, 2026 21:57

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot was unable to review this pull request because the user who requested the review has reached their quota limit.

@coveralls

coveralls commented Jun 29, 2026

Copy link
Copy Markdown

Coverage Report for CI Build 28406539470

Coverage decreased (-0.002%) to 78.806%

Details

  • Coverage decreased (-0.002%) from the base build.
  • Patch coverage: No coverable lines changed in this PR.
  • No coverage regressions found.

Uncovered Changes

No uncovered changes found.

Coverage Regressions

No coverage regressions found.


Coverage Stats

Coverage Status
Relevant Lines: 12204
Covered Lines: 10059
Line Coverage: 82.42%
Relevant Branches: 7028
Covered Branches: 5097
Branch Coverage: 72.52%
Branches in Coverage %: Yes
Coverage Strength: 422.9 hits per line

💛 - Coveralls

Signed-off-by: ferhat elmas <elmas.ferhat@gmail.com>
@ferhatelmas ferhatelmas force-pushed the ferhat/lock-delete-count branch from 7b29cd9 to 665fbe5 Compare June 29, 2026 22:21
@ferhatelmas ferhatelmas merged commit 7a0e222 into master Jun 29, 2026
24 checks passed
@ferhatelmas ferhatelmas deleted the ferhat/lock-delete-count branch June 29, 2026 22:26
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.

4 participants