search

valorem-labs-inc / clear

Valorem Clear is a DeFi money lego, enabling writing covered calls and cash-secured puts, physically settled or cash settled, American, European, and Exotic options.
https://valorem.xyz/docs/clear-overview/
Other
89 stars 12 forks source link

fix duplicate event emission #144

Closed 0xAlcibiades closed 1 year ago

0xAlcibiades commented 1 year ago

Somehow we were emitting the same events twice. This fixes.

codecov-commenter commented 1 year ago

Codecov Report

Merging #144 (375d178) into audit-fixes (c8b1d62) will decrease coverage by 0.06%. The diff coverage is 100.00%.

@@               Coverage Diff               @@
##           audit-fixes     #144      +/-   ##
===============================================
- Coverage        88.72%   88.66%   -0.07%     
===============================================
  Files                2        2              
  Lines              346      344       -2     
  Branches            54       54              
===============================================
- Hits               307      305       -2     
  Misses              30       30              
  Partials             9        9
Impacted Files Coverage Δ
src/OptionSettlementEngine.sol 99.07% <100.00%> (-0.01%) :arrow_down:

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.