Lack of detail in errors

[julien-devatom]

https://github.com/morpho-labs/morpho-blue-metamorpho/blob/eb2d02b1faee86abe6072cc72212616af7a3c554/src/MetaMorpho.sol#L247

Here, you need to find out which market is reverting. Custom errors have been implemented for that: You can add parameters to the error (check doc here) So I suggest to use custom errors here and providing the first market that is reverting.

[Rubilmax]

Agreed, but then let's use custom errors everywhere... And we roll back conventions. Agility!

[julien-devatom]

For this specific use case, I'm in favor. In blue, we are never handling lists or SG like that so its ok.

[Rubilmax]

And also blue is a "primitive"

[MerlinEgalite]

Seen in a call with @Rubilmax let's do that. Too bad it's not synchronized with bundlers.