Skip to content
This repository has been archived by the owner on Dec 2, 2024. It is now read-only.

ContractModel: Add wiggle room options to NoLockedFunds #314

Merged
merged 1 commit into from
Feb 16, 2022

Conversation

MaximilianAlgehed
Copy link
Contributor

@MaximilianAlgehed MaximilianAlgehed commented Feb 14, 2022

Some contracts, like the Uniswap example contract, have predictable overhead and rounding errors that prevent the strict NoLockedFunds property from passing. This PR introduces a little bit of wiggle room to allow for such contracts.

Pre-submit checklist:

  • Branch
    • Tests are provided (if possible)
    • Commit sequence broadly makes sense
    • Key commits have useful messages
    • Relevant tickets are mentioned in commit messages
    • Formatting, materialized Nix files, PNG optimization, etc. are updated
  • PR
    • Self-reviewed the diff
    • Useful pull request description
    • Reviewer requested

@MaximilianAlgehed MaximilianAlgehed changed the title ContractModel: added wiggle room options to NoLockedFunds ContractModel: Add wiggle room options to NoLockedFunds Feb 14, 2022
@MaximilianAlgehed MaximilianAlgehed force-pushed the PR-NLFP-WiggleRoom branch 2 times, most recently from 82f913d to fc52a44 Compare February 14, 2022 15:23
@MaximilianAlgehed
Copy link
Contributor Author

@sjoerdvisscher Do you mind looking at this when you have a gap in your schedule?

@MaximilianAlgehed
Copy link
Contributor Author

Done :)

@sjoerdvisscher sjoerdvisscher merged commit 7ef5b0a into IntersectMBO:main Feb 16, 2022
@MaximilianAlgehed MaximilianAlgehed deleted the PR-NLFP-WiggleRoom branch February 16, 2022 14:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants