Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

N-04 Unused Code #1922

Merged

Conversation

sparrowDom
Copy link
Member

The ComposablePoolExitKind is now used in #1916 and for that reason shouldn't be removed

Audit note:
Throughout the codebase, there are several instances of unused code:

  • The _fromPoolAsset(address poolAsset, uint256 poolAmount) function is
    unused.
  • The _getBPTExpected(address _asset, uint256 _amount) function is
    unused.
  • The IBalancerVault.ComposablePoolExitKind enum is unused.
    Consider reviewing each of these instances and updating or removing them in order to improve
    the clarity of the codebase.

Copy link

github-actions bot commented Nov 8, 2023

Warnings
⚠️ 👀 This PR needs at least 2 reviewers

Generated by 🚫 dangerJS against a7ba156

Base automatically changed from sparrowDom/N-03-nameing-suggestions to sparrowDom/balancer-composable-st-pool November 8, 2023 20:36
@sparrowDom sparrowDom temporarily deployed to preview-ousd-sparrowdom-0kfdpn November 8, 2023 20:38 Inactive
@sparrowDom sparrowDom merged commit 4d8a33a into sparrowDom/balancer-composable-st-pool Nov 8, 2023
@sparrowDom sparrowDom deleted the sparrowDom/N-04-unused-code branch November 8, 2023 20:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants