Skip to content

Commit

Permalink
Migrate SDK consts to Registry and use new registry utilities (#3615)
Browse files Browse the repository at this point in the history
### Description

- Remove most consts from SDK
- Refactor tests that relied on chainMetadata
- Remove JSON imports/exports in the SDK
- Update the HelloWorld package to read metadata from the registry lib
- Update the Infra package to read/write to a LocalRegistry
- Update the CLI to read and write from registries

### Drive-by changes

- Fix typo in `EvmHypCollateralVault` token standard 
- Use consistent, newest yaml lib version

### Related issues

Fixes hyperlane-xyz/issues#917
Fixes #2810

### Backward compatibility

No: 
- Several SDK exports like `chainMetadata` have been removed
- Common args for the CLI like `--chain` have been replaced with `--registry` and `--overrides`
  • Loading branch information
jmrossy authored May 1, 2024
1 parent 2928791 commit 3528b28
Show file tree
Hide file tree
Showing 204 changed files with 2,906 additions and 5,456 deletions.
2 changes: 1 addition & 1 deletion .changeset/khaki-days-float.md
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
---
"@hyperlane-xyz/sdk": patch
'@hyperlane-xyz/sdk': patch
---

Allow gasLimit overrides in the SDK/CLI for deploy txs
5 changes: 5 additions & 0 deletions .changeset/moody-colts-dress.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@hyperlane-xyz/sdk': minor
---

Remove consts such as chainMetadata from SDK
5 changes: 5 additions & 0 deletions .changeset/nine-masks-guess.md
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@hyperlane-xyz/cli': minor
---

Restructure CLI params around registries
100 changes: 0 additions & 100 deletions .github/workflows/cron.yml

This file was deleted.

2 changes: 1 addition & 1 deletion .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
node-version: 18.x

- name: Install Dependencies
run: yarn
run: yarn install --immutable

- name: Create Release PR or Publish to NPM
id: changesets
Expand Down
1 change: 1 addition & 0 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,6 +18,7 @@ env:
LOG_FORMAT: PRETTY
CARGO_TERM_COLOR: always
RUST_BACKTRACE: full
REGISTRY_URI: ../../node_modules/@hyperlane-xyz/registry/dist

jobs:
yarn-install:
Expand Down
Loading

0 comments on commit 3528b28

Please sign in to comment.