Skip to content

Conversation

@godzillaba
Copy link
Contributor

@godzillaba godzillaba commented Jul 29, 2025

bump sdk to v4 and do bare minimum migration

@cla-bot cla-bot bot added the cla-signed label Jul 29, 2025
This was referenced Jul 29, 2025
@godzillaba godzillaba marked this pull request as ready for review August 1, 2025 17:40
Copy link
Contributor

@TucksonDev TucksonDev left a comment

Choose a reason for hiding this comment

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

LGTM. Adding a minor, not blocking, comment

Copy link
Member

@gzeoneth gzeoneth left a comment

Choose a reason for hiding this comment

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

LGTM

@gzeoneth gzeoneth merged commit 9283970 into main Aug 5, 2025
10 checks passed
gzeoneth added a commit that referenced this pull request Aug 18, 2025
* nits on README

add link to LICENSE, replaced dash with semicolon next to Discord

* Stable foundry (#106)

* use stable foundry

* fix

* Use template repo's signature and storage test scripts (#107)

* use stable foundry

* fix

* use template scripts

* run new scripts

---------

Co-authored-by: gzeon <im@gzeon.dev>

* Bump sdk (#105)

* bump sdk

* find and replace

* write new network format (#109)

* chore: isTestnet true

* chore: set version to 1.2.4

---------

Co-authored-by: gzeon <hng@offchainlabs.com>
Co-authored-by: gzeon <im@gzeon.dev>

* Add missing nativeToken property to local network (#110)

* add missing nativeToken property to local network

* do not scale fee amount in test since sdk now handles

* chore: bump version to 1.2.5

---------

Co-authored-by: gzeon <im@gzeon.dev>

---------

Co-authored-by: gzeon <hng@offchainlabs.com>
Co-authored-by: Allan <agulley@offchainlabs.com>
Co-authored-by: Henry <11198460+godzillaba@users.noreply.github.com>
Co-authored-by: gzeon <im@gzeon.dev>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants