Skip to content

Conversation

gzeoneth
Copy link
Member

No description provided.

@gzeoneth gzeoneth marked this pull request as ready for review July 11, 2025 09:06
Copy link

@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.

Requesting one change, or at least, confirm that types should be different

/// @notice Available in ArbOS version 30 and above
/// @param gas amount of gas paid in increments of 256 when not the program is not cached
/// @param cached amount of gas paid in increments of 64 when the program is cached
function setWasmMinInitGas(uint8 gas, uint16 cached) external;

Choose a reason for hiding this comment

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

Potential type mismatch. Both gas and cached are marked as uint64 in the implementation file (here).

gzeoneth and others added 5 commits July 30, 2025 01:12
* refactor: remove arbos 41 methods

* refactor: remove arbos 40 methods

* refactor: remove arbos 31 methods

* refactor: remove arbos 30 methods

* refactor: remove arbos 20 methods

* refactor: remove arbos 11 methods

* refactor: remove arbos 10 methods

* refactor: remove arbos 5 methods

* docs: add LICENSE.md

* chore: set up repo and ci

* docs: add README.md

* ci: push

* ci: fix wrong path

* chore: commit yarn lock

* format: fix

* chore: remove ChainOwnerRectified
* refactor: remove arbos 41 methods

* refactor: remove arbos 40 methods

* refactor: remove arbos 31 methods

* refactor: remove arbos 30 methods

* refactor: remove arbos 20 methods

* refactor: remove arbos 11 methods

* refactor: remove arbos 10 methods

* refactor: remove arbos 5 methods

* docs: add LICENSE.md

* chore: set up repo and ci

* docs: add README.md

* ci: push

* ci: fix wrong path

* chore: commit yarn lock

* feat: add arbos 5 methods

* format: fix

* chore: 5.0.0

* chore: remove ChainOwnerRectified
* refactor: remove arbos 41 methods

* refactor: remove arbos 40 methods

* refactor: remove arbos 31 methods

* refactor: remove arbos 30 methods

* refactor: remove arbos 20 methods

* refactor: remove arbos 11 methods

* refactor: remove arbos 10 methods

* refactor: remove arbos 5 methods

* docs: add LICENSE.md

* chore: set up repo and ci

* docs: add README.md

* ci: push

* ci: fix wrong path

* chore: commit yarn lock

* feat: add arbos 5 methods

* feat: add arbos 10 methods

* format: fix

* chore: 5.0.0

* chore: 10.0.0

* chore: remove ChainOwnerRectified
* refactor: remove arbos 41 methods

* refactor: remove arbos 40 methods

* refactor: remove arbos 31 methods

* refactor: remove arbos 30 methods

* refactor: remove arbos 20 methods

* refactor: remove arbos 11 methods

* refactor: remove arbos 10 methods

* refactor: remove arbos 5 methods

* docs: add LICENSE.md

* chore: set up repo and ci

* docs: add README.md

* ci: push

* ci: fix wrong path

* chore: commit yarn lock

* feat: add arbos 5 methods

* feat: add arbos 10 methods

* feat: add arbos 11 methods

* format: fix

* chore: 5.0.0

* chore: 10.0.0

* chore: 11.0.0

* chore: remove ChainOwnerRectified

* feat: ChainOwnerRectified
@gzeoneth gzeoneth merged commit 1592517 into arbos-20 Jul 29, 2025
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants