uutils / platform-info

A cross-platform way to get information about your machine
MIT License
85 stars 25 forks source link

Coverage & Fix a windows clippy warning (set_len) #18

Closed sylvestre closed 2 years ago

sylvestre commented 2 years ago
calling `set_len()` immediately after reserving a buffer creates uninitialized values
codecov[bot] commented 2 years ago

Codecov Report

Merging #18 (aeaff6d) into master (a356060) will increase coverage by 28.49%. The diff coverage is 83.33%.

:exclamation: Current head aeaff6d differs from pull request most recent head 3ab859a. Consider uploading reports for the commit 3ab859a to get more accurate results

@@             Coverage Diff             @@
##           master      #18       +/-   ##
===========================================
+ Coverage   16.08%   44.58%   +28.49%     
===========================================
  Files           4        4               
  Lines         230      240       +10     
  Branches       36       37        +1     
===========================================
+ Hits           37      107       +70     
+ Misses        193      119       -74     
- Partials        0       14       +14     
Flag Coverage Δ
macos_latest 89.74% <ø> (?)
ubuntu_latest 89.74% <ø> (?)
windows_latest 39.90% <83.33%> (?)

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

Impacted Files Coverage Δ
tests/integration_test.rs 85.71% <81.81%> (-8.41%) :arrow_down:
src/windows.rs 33.15% <100.00%> (+33.15%) :arrow_up:
src/unix.rs 90.47% <0.00%> (-4.77%) :arrow_down:

Continue to review full report at Codecov.

Legend - Click here to learn more Δ = absolute <relative> (impact), ø = not affected, ? = missing data Powered by Codecov. Last update a356060...3ab859a. Read the comment docs.