Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: multi collateral warp routes #3820

Merged
merged 20 commits into from
May 23, 2024
Merged

feat: multi collateral warp routes #3820

merged 20 commits into from
May 23, 2024

Conversation

yorhodes
Copy link
Member

@yorhodes yorhodes commented May 20, 2024

Description

Adds multicollateral warp route support to the CLI. Dedupes SDK warp route types and zod schemas (via type inference).

Drive-by changes

  • Removes scattered ICA logic from deployers in favor of config time resolution

Related issues

Backward compatibility

  • No

Testing

  • E2E Tests

Copy link

changeset-bot bot commented May 20, 2024

🦋 Changeset detected

Latest commit: 24201d8

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 7 packages
Name Type
@hyperlane-xyz/infra Minor
@hyperlane-xyz/cli Minor
@hyperlane-xyz/sdk Minor
@hyperlane-xyz/helloworld Minor
@hyperlane-xyz/core Minor
@hyperlane-xyz/ccip-server Minor
@hyperlane-xyz/utils Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@yorhodes yorhodes changed the title Ensure dir exists before reading in fixture tests feat: multi collateral warp routes May 21, 2024
@yorhodes yorhodes marked this pull request as ready for review May 21, 2024 20:35
Copy link

codecov bot commented May 21, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 63.96%. Comparing base (b22a0f4) to head (24201d8).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #3820   +/-   ##
=======================================
  Coverage   63.96%   63.96%           
=======================================
  Files         124      124           
  Lines        1543     1543           
  Branches      167      167           
=======================================
  Hits          987      987           
  Misses        530      530           
  Partials       26       26           
Components Coverage Δ
core 84.61% <ø> (ø)
hooks 67.50% <ø> (ø)
isms 39.52% <ø> (ø)
token 65.41% <ø> (ø)
middlewares 77.05% <ø> (ø)

@yorhodes yorhodes requested review from ltyu and removed request for tkporter, aroralanuk, Skunkchain and avious00 May 21, 2024 22:00
Copy link
Contributor

@jmrossy jmrossy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice cleanup here, just a few questions and comments

typescript/cli/src/config/warp.ts Outdated Show resolved Hide resolved
typescript/cli/src/deploy/warp.ts Outdated Show resolved Hide resolved
typescript/sdk/src/deploy/types.ts Show resolved Hide resolved
typescript/sdk/src/token/checker.ts Outdated Show resolved Hide resolved
typescript/sdk/src/token/config.ts Show resolved Hide resolved
typescript/sdk/src/token/schemas.ts Outdated Show resolved Hide resolved
typescript/sdk/src/token/types.ts Show resolved Hide resolved
Copy link
Contributor

@ltyu ltyu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks pretty good upon initial review. just some quick questions. will continue later

typescript/sdk/src/token/schemas.ts Outdated Show resolved Hide resolved
typescript/sdk/src/token/schemas.ts Outdated Show resolved Hide resolved
typescript/sdk/src/token/schemas.ts Outdated Show resolved Hide resolved
@yorhodes yorhodes requested review from ltyu and jmrossy May 22, 2024 23:28
@yorhodes yorhodes enabled auto-merge (squash) May 23, 2024 17:00
@yorhodes yorhodes merged commit 39ea7cd into main May 23, 2024
39 checks passed
@yorhodes yorhodes deleted the sdk-patch branch May 23, 2024 17:20
github-merge-queue bot pushed a commit that referenced this pull request Jun 19, 2024
### Description

Token deployer should always derive token metadata's total supply to 0,
otherwise the collateral token's total supply will be minted initially
on the synthetic, creating 2x the desired total supply.

### Related issues

- Regression introduced by
#3820
- opens #4013

### Backward compatibility

Yes

### Testing

Unit Tests
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Support Multi-Collateral Warp Route deployment in CLI
3 participants