XRPLF / rippled

Decentralized cryptocurrency blockchain daemon implementing the XRP Ledger protocol in C++
https://xrpl.org
ISC License
4.53k stars 1.47k forks source link

fix AMMClawback vote behavior to default NO #5181

Closed yinyiqian1 closed 2 weeks ago

yinyiqian1 commented 2 weeks ago

High Level Overview of Change

Context of Change

Type of Change

API Impact

codecov[bot] commented 2 weeks ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 77.7%. Comparing base (c5c0e70) to head (ab77a92). Report is 1 commits behind head on develop.

Additional details and impacted files [![Impacted file tree graph](https://app.codecov.io/gh/XRPLF/rippled/pull/5181/graphs/tree.svg?width=650&height=150&src=pr&token=i2RPGI5xGF&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=XRPLF)](https://app.codecov.io/gh/XRPLF/rippled/pull/5181?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=XRPLF) ```diff @@ Coverage Diff @@ ## develop #5181 +/- ## ======================================= Coverage 77.7% 77.7% ======================================= Files 779 779 Lines 66015 66016 +1 Branches 8156 8152 -4 ======================================= + Hits 51261 51270 +9 + Misses 14754 14746 -8 ``` [see 4 files with indirect coverage changes](https://app.codecov.io/gh/XRPLF/rippled/pull/5181/indirect-changes?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=XRPLF) [![Impacted file tree graph](https://app.codecov.io/gh/XRPLF/rippled/pull/5181/graphs/tree.svg?width=650&height=150&src=pr&token=i2RPGI5xGF&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=XRPLF)](https://app.codecov.io/gh/XRPLF/rippled/pull/5181?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=XRPLF)
mvadari commented 2 weeks ago

This was rolled into #5103 so is no longer needed as a separate PR.