Skip to content

Add assert to NpgsqlCommand.Transaction if it's completed#6151

Merged
vonzshik merged 3 commits into
mainfrom
6149-command-transaction-assert-completed
Jul 2, 2025
Merged

Add assert to NpgsqlCommand.Transaction if it's completed#6151
vonzshik merged 3 commits into
mainfrom
6149-command-transaction-assert-completed

Conversation

@vonzshik

@vonzshik vonzshik commented Jul 1, 2025

Copy link
Copy Markdown
Contributor

Closes #6149

@vonzshik vonzshik requested a review from roji as a code owner July 1, 2025 12:55

@roji roji 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.

Add a quick test maybe?

@vonzshik vonzshik enabled auto-merge (squash) July 2, 2025 12:11
@vonzshik vonzshik merged commit 3dae121 into main Jul 2, 2025
14 checks passed
@vonzshik vonzshik deleted the 6149-command-transaction-assert-completed branch July 2, 2025 12:16
This was referenced Nov 23, 2025
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.

Using a completed NpgsqlTransaction silently causes command to execute without transaction

3 participants