Skip to content
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

Fix default initialization for various types #104

Merged
merged 1 commit into from
Oct 24, 2024

Conversation

Schamper
Copy link
Member

Fixes #102 and the bug that occurred in fox-it/dissect.hypervisor#31.

Copy link

codecov bot commented Oct 22, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 92.46%. Comparing base (09830d2) to head (79d1698).
Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #104      +/-   ##
==========================================
+ Coverage   92.12%   92.46%   +0.33%     
==========================================
  Files          20       20              
  Lines        2146     2176      +30     
==========================================
+ Hits         1977     2012      +35     
+ Misses        169      164       -5     
Flag Coverage Δ
unittests 92.46% <100.00%> (+0.33%) ⬆️

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.

@Schamper Schamper force-pushed the fix-default-initialization branch 3 times, most recently from 64b3675 to 6b45db8 Compare October 22, 2024 20:52
@Schamper Schamper force-pushed the fix-default-initialization branch 2 times, most recently from f77e77c to f8503a4 Compare October 23, 2024 14:13
@Schamper Schamper requested a review from twiggler October 23, 2024 14:15
@Schamper Schamper force-pushed the fix-default-initialization branch 2 times, most recently from cd2ad25 to a31c4e7 Compare October 23, 2024 14:38
@Schamper Schamper force-pushed the fix-default-initialization branch from a31c4e7 to 79d1698 Compare October 23, 2024 19:43
Copy link

@twiggler twiggler left a comment

Choose a reason for hiding this comment

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

Nice!

Lked the "concept checks" like is_readable_type.

@Schamper Schamper merged commit b5ce35d into main Oct 24, 2024
20 checks passed
@Schamper Schamper deleted the fix-default-initialization branch October 24, 2024 08:44
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.

Errors when initialising a struct with partial keywords or arguments
2 participants