Skip to content

fix(hc): Add missing using #54188

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Aug 4, 2023
Merged

fix(hc): Add missing using #54188

merged 1 commit into from
Aug 4, 2023

Conversation

mrduncan
Copy link
Member

@mrduncan mrduncan commented Aug 4, 2023

This using is required so that the transaction is opened to the correct database when running in split-db mode.

This using is required so that the transaction is opened to the correct
database.
@codecov
Copy link

codecov bot commented Aug 4, 2023

Codecov Report

Merging #54188 (65529a3) into master (2aa6c18) will decrease coverage by 0.01%.
Report is 7 commits behind head on master.
The diff coverage is n/a.

Additional details and impacted files
@@            Coverage Diff             @@
##           master   #54188      +/-   ##
==========================================
- Coverage   79.63%   79.62%   -0.01%     
==========================================
  Files        4982     4982              
  Lines      211102   211102              
  Branches    35961    35961              
==========================================
- Hits       168104   168098       -6     
- Misses      37831    37838       +7     
+ Partials     5167     5166       -1     

see 7 files with indirect coverage changes

@mrduncan mrduncan marked this pull request as ready for review August 4, 2023 18:43
@mrduncan mrduncan requested a review from a team August 4, 2023 18:44
@mrduncan mrduncan merged commit 388b444 into master Aug 4, 2023
@mrduncan mrduncan deleted the mrduncan/missing-using branch August 4, 2023 19:22
@github-actions github-actions bot locked and limited conversation to collaborators Aug 20, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants