Skip to content

Conversation

@Schamper
Copy link
Member

Fixes #105

@codecov
Copy link

codecov bot commented Oct 24, 2024

Codecov Report

Attention: Patch coverage is 82.14286% with 15 lines in your changes missing coverage. Please review.

Project coverage is 92.33%. Comparing base (ed1daf2) to head (063b1c8).
Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
dissect/cstruct/parser.py 78.57% 3 Missing ⚠️
dissect/cstruct/types/base.py 78.57% 3 Missing ⚠️
dissect/cstruct/utils.py 80.00% 3 Missing ⚠️
dissect/cstruct/compiler.py 71.42% 2 Missing ⚠️
dissect/cstruct/cstruct.py 80.00% 2 Missing ⚠️
dissect/cstruct/types/structure.py 75.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #106      +/-   ##
==========================================
- Coverage   92.46%   92.33%   -0.13%     
==========================================
  Files          20       20              
  Lines        2176     2180       +4     
==========================================
+ Hits         2012     2013       +1     
- Misses        164      167       +3     
Flag Coverage Δ
unittests 92.33% <82.14%> (-0.13%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Horofic
Horofic previously approved these changes Jan 17, 2025
@Schamper Schamper merged commit e51e001 into main Jan 20, 2025
18 of 20 checks passed
@Schamper Schamper deleted the ruff branch January 20, 2025 09:21
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.

Change linter to Ruff

3 participants