crowdsec icon indicating copy to clipboard operation
crowdsec copied to clipboard

pkg/dumps.DumpTree: split to reduce complexity

Open mmetc opened this issue 1 year ago • 2 comments

mmetc avatar May 13 '24 11:05 mmetc

@mmetc: There are no area labels on this PR. You can add as many areas as you see fit.

  • /area agent
  • /area local-api
  • /area cscli
  • /area appsec
  • /area security
  • /area configuration
Details

I am a bot created to help the crowdsecurity developers manage community feedback and contributions. You can check out my manifest file to understand my behavior and what I can do. If you want to use this for your project, you can check out the BirthdayResearch/oss-governance-bot repository.

github-actions[bot] avatar May 13 '24 11:05 github-actions[bot]

Codecov Report

Attention: Patch coverage is 85.71429% with 6 lines in your changes are missing coverage. Please review.

Project coverage is 52.37%. Comparing base (b8ee31a) to head (b0ced8e). Report is 1 commits behind head on master.

Files Patch % Lines
pkg/dumps/parser_dump.go 85.71% 5 Missing and 1 partial :warning:
Additional details and impacted files
@@            Coverage Diff             @@
##           master    #3001      +/-   ##
==========================================
- Coverage   52.37%   52.37%   -0.01%     
==========================================
  Files         280      280              
  Lines       35422    35431       +9     
==========================================
+ Hits        18554    18557       +3     
- Misses      15027    15032       +5     
- Partials     1841     1842       +1     
Flag Coverage Δ
bats 40.09% <90.00%> (+0.03%) :arrow_up:
unit-linux 34.84% <0.00%> (-0.03%) :arrow_down:
unit-windows 30.97% <0.00%> (-0.03%) :arrow_down:

Flags with carried forward coverage won't be shown. Click here to find out more.

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

codecov-commenter avatar May 21 '24 08:05 codecov-commenter