You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I think on the (off-chain) tooling side it would be useful to have the possibility to enable bubbling up the revert reasons for the create built-ins such as create_minimal_proxy_to, create_copy_of, create_from_blueprint, and the planned raw_create built-in here. This means you could add a kwarg to these functions like raw_revert_data: uint256 = 0 and if it's above 0 the functions also return some return_data with Bytes[raw_revert_data] that can be processed further in case of a failure. If revert_on_failure is True, the called revert statement should be like raw_revert(return_data).
The text was updated successfully, but these errors were encountered:
I think on the (off-chain) tooling side it would be useful to have the possibility to enable bubbling up the revert reasons for the
create
built-ins such ascreate_minimal_proxy_to
,create_copy_of
,create_from_blueprint
, and the plannedraw_create
built-in here. This means you could add a kwarg to these functions likeraw_revert_data: uint256 = 0
and if it's above0
the functions also return somereturn_data
withBytes[raw_revert_data]
that can be processed further in case of a failure. Ifrevert_on_failure
isTrue
, the calledrevert
statement should be likeraw_revert(return_data)
.The text was updated successfully, but these errors were encountered: