Skip to content

Conversation

@Brownjy
Copy link
Collaborator

@Brownjy Brownjy commented Dec 28, 2023

No description provided.

@codecov
Copy link

codecov bot commented Dec 28, 2023

Codecov Report

Attention: 58 lines in your changes are missing coverage. Please review.

Comparison is base (9a68869) 64.34% compared to head (2314f75) 64.42%.
Report is 1 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main      #76      +/-   ##
==========================================
+ Coverage   64.34%   64.42%   +0.08%     
==========================================
  Files         104      104              
  Lines       11389    11574     +185     
==========================================
+ Hits         7328     7457     +129     
- Misses       3342     3384      +42     
- Partials      719      733      +14     
Files Coverage Δ
block/blocktest/adapter.go 97.82% <100.00%> (+0.34%) ⬆️
block/local/walker.go 52.34% <ø> (ø)
block/local/adapter.go 48.64% <75.00%> (+0.73%) ⬆️
block/transient/adapter.go 0.00% <0.00%> (ø)
controller/repository_ctl.go 79.27% <27.27%> (-1.66%) ⬇️
block/gs/adapter.go 39.95% <66.66%> (+1.59%) ⬆️
block/mem/adapter.go 9.71% <0.00%> (-0.33%) ⬇️
block/s3/adapter.go 42.07% <70.27%> (+1.63%) ⬆️
block/azure/adapter.go 43.20% <48.27%> (+0.34%) ⬆️

... and 1 file with indirect coverage changes

Copy link
Collaborator

@hunjixin hunjixin left a comment

Choose a reason for hiding this comment

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

LGTM

@hunjixin hunjixin merged commit 5ac70cf into main Jan 11, 2024
@hunjixin hunjixin deleted the feat/delete_repo_data branch January 11, 2024 02:15
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.

3 participants