fix: resolve duplicate charge_fee_to_insurance from squash merge - #83
Conversation
…bt_to_insurance The squash of PR #82 landed two definitions with the same name: - 2-arg: charge_fee_to_insurance(idx, fee) -> Result<(u128,u128)> [new, from fix/PERC-8463] - 1-arg: charge_fee_to_insurance(idx) -> u128 [old sweep-debt helper] Rename the 1-arg variant to sweep_fee_debt_to_insurance (matching its actual role: sweeping existing negative fee_credits into insurance). Update its two call sites in settle_maintenance_fee and settle_maintenance_fee_best_effort_for_crank. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
|
Warning Rate limit exceeded
Your organization is not enrolled in usage-based pricing. Contact your admin to enable usage-based pricing to continue reviews beyond the rate limit, or try again in 3 minutes and 39 seconds. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Summary
impl RiskEnginemethods with the same namecharge_fee_to_insurancebut different signatures, causing a compile errorsweep_fee_debt_to_insurance(what it actually does)settle_maintenance_feeandsettle_maintenance_fee_best_effort_for_crankTest plan
cargo test --libpassescargo build-sbf --features mainnetin percolator-prog compiles cleanly🤖 Generated with Claude Code