search

morpho-org / morpho-optimizers

Core contracts of Morpho V1 Optimizers.
https://app.morpho.org
GNU Affero General Public License v3.0
137 stars 22 forks source link

fix: removing useless lines #1664

Closed Tristan22400 closed 1 year ago

Tristan22400 commented 1 year ago

Pull Request

Issue(s) fixed

This pull request fixes #to-complete

or

This pull request fixes:

Tristan22400 commented 1 year ago

I do not understand why it did not fail before.

Rubilmax commented 1 year ago

I do not understand why it did not fail before.

It's a matter of forge version ^^ https://github.com/foundry-rs/foundry/pull/4827

makcandrov commented 1 year ago

Nice catch! Isn't there a check for Compound tests? Because the same problem exists in some Compound tests (cf here)

Tristan22400 commented 1 year ago

Are you sure this is related?

yes, at least one of the problems. image