[#824] General cleanup
Description
While working on #824 (closed), I happened to come across a few unrelated things that needed to be cleaned up and a bug:
- We should enable the
InstanceSigs
extension globally - In the
Pure
module, we were usingText
where we should have been using theAlias
data type. - We had no tests for cleveland's
resolveAddress
- Bug: In
Pure
, when a contract fails, we were enriching theExecutorError
with the contract's alias - however, theexceptionHandler
was discarding this data, so the contract's alias never showed up in error messages as intended.
This MR fixes all of the above.
Related issue(s)
Resolves part of #824 (closed)
✅ Checklist for your Merge Request
Related changes (conditional)
-
Tests (see short guidelines)
-
If I added new functionality, I added tests covering it. -
If I fixed a bug, I added a regression test to prevent the bug from silently reappearing again.
-
-
Documentation
Stylistic guide (mandatory)
-
My commits comply with the following policy. -
My code complies with the style guide.
Edited by Diogo Castro