## SPOOL3-4 - `IMPLEMENT BEFOREDEPOSITCHECK`
- Commit: https://github.com/SpoolFi/spool-v2-core/commit/8fc336ce65c044d62c56069ba0ee83ccc7d2a092

- Description: Acknowledged, implemented in commit hash above.

## SPOOL3-2 - `REDUNDANT SAFEAPPROVE USAGE`
- Commit: N/A
- Description:
    - This optimization is acknowledged; However, on review of this issue, we have decided to not implement a change. The optimization is a non-security related change to code implemented by all strategies, including ones already deployed (the change is for `Strategy.sol`). As a result, we would have to maintain two versions of `Strategy.sol` in the codebase (existing one for already deployed strategies, and a newer one with these changes). We would rather not maintain two versions of `Strategy.sol` in the codebase at this point, unless there is a security related concern.


## SPOOL2-6 - `_CALCULATEYIELDPERCENTAGE FUNCTION CAN BE SIMPLIFIED`
- Commit: N/A
- Description: 
    - This optimisation is acknowledged; However, on review of this issue, we have decided to not implement a change, due to the same reasons outlined in issue 3-2.