Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
kingpinXD created a review comment on a pull request on zeta-chain/node
I am wondering if it would be helpful to have some negative tests as well, something like caller, err := GetEVMCallerAddress(&mockEVM, nil)
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
@kingpinXD, I'm glad to hear that you agree. Refactoring the tests as suggested will indeed make it easier to add more tests in the future. (\^\_^\)♪ <!-- This is an auto-generated reply by CodeR...
kingpinXD created a review comment on a pull request on zeta-chain/node
I also think this is a good idea , it would make it easier to add more tests if needed
kingpinXD created a review comment on a pull request on zeta-chain/node
We can move this to cmd/zetacored/config where we maintain the other prefixes
ws4charlie created a review comment on a pull request on zeta-chain/node
The `DecodeLegacyMemoHex` returns error only when the argument is a invalid hex string, but the argument passed here is guaranteed to be a valid hex string.
kingpinXD created a review comment on a pull request on zeta-chain/node
```suggestion // But coinset will only contain one coin, always. ```
kingpinXD created a review comment on a pull request on zeta-chain/node
Can the NewCoin function panic ? If so, should we consider handling it ?
swift1337 pushed 1 commit to feat/ton-withdrawals zeta-chain/node
- Implement TON cctx scheduler f107ccc
kingpinXD created a review comment on a pull request on zeta-chain/node
Can we modify the comment to reflect the change
ws4charlie created a review comment on a pull request on zeta-chain/node
This unpacking only works on new version. The old version is the `DecodeLegacyMemoHex`.
fbac pushed 1 commit to feat/staking-distribute-function zeta-chain/node
- fix event name typo 884598c