Use module paths instead of conditional module includes for sys modules

[joshka]

Saves 1 LOC

[joshka]

Hrm, this one seems to be trigger a lot of build failures. Maybe too many to worry about a single LOC :D

[ilai-deutel]

This is a great idea, and regardless of line count it also looks cleaner. I don't understand why the dead code warnings appear though, and only on Windows not Linux or Mac.

I don't know if it's an issue with the code or with the compiler, I may do some more testing with a Windows machine later :)

[ilai-deutel]

I just figured it out, I think the #[cfg(windows)] mod windows; line needs to be removed

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 48.41%. Comparing base (2a5840e) to head (6b40a4d).
Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #330      +/-   ##
==========================================
+ Coverage   48.33%   48.41%   +0.08%     
==========================================
  Files          10       10              
  Lines        1138     1138              
==========================================
+ Hits          550      551       +1     
+ Misses        588      587       -1     

Flag	Coverage Δ
wasm32-wasip1	49.00% <ø> (ø)
x86_64-apple-darwin	49.09% <ø> (?)
x86_64-pc-windows-gnu	48.29% <ø> (ø)
x86_64-unknown-linux-gnu	49.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.

---

🚨 Try these New Features:

• Flaky Tests Detection - Detect and resolve failed and flaky tests

[ilai-deutel]

FYI, I reported the misleading warning: rust-lang/rust#133197