Skip to content

Conversation

@Miauwkeru
Copy link
Contributor

Updates the loader and child plugin according to
https://github.com/fox-it/dissect.hypervisor/pull/71/files

@Miauwkeru Miauwkeru requested a review from Schamper January 28, 2026 15:45
Schamper
Schamper previously approved these changes Jan 29, 2026
Copy link
Member

@Schamper Schamper left a comment

Choose a reason for hiding this comment

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

Snapshots will be added in a next PR once the VMDK rewrite is merged.

@codecov
Copy link

codecov bot commented Jan 29, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 80.75%. Comparing base (4eeb069) to head (1ce3096).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1523   +/-   ##
=======================================
  Coverage   80.75%   80.75%           
=======================================
  Files         396      396           
  Lines       34750    34750           
=======================================
  Hits        28063    28063           
  Misses       6687     6687           
Flag Coverage Δ
unittests 80.75% <100.00%> (ø)

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.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed-hq
Copy link

codspeed-hq bot commented Jan 29, 2026

Merging this PR will not alter performance

✅ 11 untouched benchmarks


Comparing virtualbox-update (1ce3096) with main (4eeb069)

Open in CodSpeed

@Miauwkeru Miauwkeru merged commit f787f5a into main Jan 29, 2026
22 of 24 checks passed
@Miauwkeru Miauwkeru deleted the virtualbox-update branch January 29, 2026 13:27
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