Skip to content

Register the Connection and Transaction provider interfaces#3668

Merged
lillo42 merged 5 commits into
BrighterCommand:masterfrom
lillo42:register-connection-and-transaction
Jul 17, 2025
Merged

Register the Connection and Transaction provider interfaces#3668
lillo42 merged 5 commits into
BrighterCommand:masterfrom
lillo42:register-connection-and-transaction

Conversation

@lillo42

@lillo42 lillo42 commented Jul 15, 2025

Copy link
Copy Markdown
Contributor

This PR is about allow Brighter to register the Connection and Transaction provider interfaces based on the provided type

I'm also align MongoDB implementation with others inbox/outbox/distributed lock implementation, by having a UnitOfWork and Connection provider

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gates Passed
4 Quality Gates Passed

See analysis details in CodeScene

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

@iancooper iancooper added 2 - In Progress v10 .NET Pull requests that update .net code labels Jul 15, 2025

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gates Failed
Enforce advisory code health rules (1 file with Code Duplication)

Gates Passed
3 Quality Gates Passed

See analysis details in CodeScene

Reason for failure
Enforce advisory code health rules Violations Code Health Impact
MongoDbUnitOfWork.cs 1 advisory rule 10.00 → 9.39 Suppress

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

Comment thread src/Paramore.Brighter.MongoDb/MongoDbUnitOfWork.cs

@codescene-delta-analysis codescene-delta-analysis Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Gates Failed
Enforce advisory code health rules (1 file with Code Duplication)

Gates Passed
3 Quality Gates Passed

See analysis details in CodeScene

Reason for failure
Enforce advisory code health rules Violations Code Health Impact
MongoDbUnitOfWork.cs 1 advisory rule 10.00 → 9.39 Suppress

Quality Gate Profile: Clean Code Collective
Want more control? Customize Code Health rules or catch issues early with our IDE extension and CLI tool.

@lillo42 lillo42 merged commit 76076f1 into BrighterCommand:master Jul 17, 2025
21 of 24 checks passed
@lillo42 lillo42 deleted the register-connection-and-transaction branch July 17, 2025 08:55
DevJonny pushed a commit to DevJonny/Brighter that referenced this pull request Feb 28, 2026
…Command#3668)

* Register the Connection and Transaction provider interfaces

* fix: test

* feat: Align mongodb inbox/outbox/distributed lock with other outbox implementation

* fx error when registering unit of work as ConnectionProvider
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3 - Done .NET Pull requests that update .net code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants