feat: support CCIP Hooks and ISMs in the SDK #5413
Conversation
|
paulbalaji
force-pushed
the
pb/ccip-sdk
branch
from
a82c1b4 to
ec73648
Compare
| // NOTE: this deploys the ism too on the destination chain if it doesn't exist | ||
| protected async deployCCIPHook({ | ||
| config, | ||
| }: { |
There was a problem hiding this comment.
the CCIP hook/ism deploy logic has been moved to the HyperlaneCCIPDeployer in the next PR #5412
| @@ -64,7 +64,7 @@ contract CCIPIsm is AbstractMessageIdAuthorizedIsm, CCIPReceiver { | |||
| preVerifyMessage(messageId, msg.value); | |||
| } | |||
|
|
|||
| function _isAuthorized() internal pure override returns (bool) { | |||
| function _isAuthorized() internal view override returns (bool) { | |||
There was a problem hiding this comment.
required to get build working
| protected async deployCCIPIsm( | ||
| destination: ChainName, | ||
| config: CCIPIsmConfig, | ||
| ): Promise<CCIPIsm> { |
There was a problem hiding this comment.
the CCIP hook/ism deploy logic has been moved to the HyperlaneCCIPDeployer in the next PR #5412
Description
feat: support CCIP Hooks and ISMs in the SDK
HyperlaneIsmFactory,HyperlaneHookDeployer,EvmHookModule)EvmIsmReader,EvmHookReader)Drive-by changes
Related issues
Backward compatibility
Testing