BrettMayson / arma-rs

GNU General Public License v2.0
42 stars 8 forks source link

Small FromArma impl fixes #46

Closed Eathox closed 1 year ago

Eathox commented 1 year ago

Various small things I noticed while working on the trait derive PR, decided to make it a separate PR

codecov[bot] commented 1 year ago

Codecov Report

Merging #46 (de4615a) into main (427671d) will increase coverage by 0.72%. Report is 3 commits behind head on main. The diff coverage is 98.30%.

@@            Coverage Diff             @@
##             main      #46      +/-   ##
==========================================
+ Coverage   89.81%   90.53%   +0.72%     
==========================================
  Files          28       28              
  Lines        2935     2928       -7     
==========================================
+ Hits         2636     2651      +15     
+ Misses        299      277      -22     
Files Changed Coverage Δ
arma-rs/src/value/mod.rs 94.91% <83.33%> (+1.81%) :arrow_up:
arma-rs/src/value/from_arma.rs 98.68% <100.00%> (+10.25%) :arrow_up:
arma-rs/src/value/into_arma.rs 98.92% <100.00%> (+0.01%) :arrow_up:
arma-rs/src/value/loadout/inventory_item.rs 93.75% <100.00%> (ø)

... and 4 files with indirect coverage changes