Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
brewmaster012 pushed 1 commit to dht-removal zeta-chain/node
- add new telemetry: 8123/connectedpeers; fix deadlock in a few other 8123 handlers 21d9d7b
julianrubino pushed 1 commit to sim-import-export zeta-chain/node
- Delete main branch from sim test workflow cf60f5a
julianrubino pushed 1 commit to sim-import-export zeta-chain/node
- Update .github/workflows/sim.yaml workflow name Co-authored-by: Alex Gartner <[email protected]> 399569d
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
_:bulb: Codebase verification_ **Whitelisted peers implementation requires validation enhancement** The implementation needs additional safeguards: - `whitelistedPeers` is populated from `Grantee...
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
_:warning: Potential issue_ **Add test coverage for whitelisted peers.** The static analysis indicates that the `whitelistedPeers` parameter usage lacks test coverage. Consider adding the follow...
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
_:warning: Potential issue_ **Enhance error handling in peer ID conversion.** The error handling could be more informative and include context about which public key failed to convert. Apply thi...
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
_:warning: Potential issue_ **Replace debug logging with structured metrics.** The current implementation has several issues: 1. Uses debug-oriented spew.Dump in production code 2. Hardcoded 10-s...
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
_:warning: Potential issue_ **Remove debug-only import.** The `go-spew` package is typically used for debugging and should not be included in production code. Consider using structured logging in...
coderabbitai[bot] created a review on a pull request on zeta-chain/node
**Actionable comments posted: 5** <details> <summary>🧹 Outside diff range and nitpick comments (3)</summary><blockquote> <details> <summary>zetaclient/tss/tss_signer.go (3)</summary><blockquote> ...
coderabbitai[bot] created a review on a pull request on zeta-chain/node
**Actionable comments posted: 5** <details> <summary>🧹 Outside diff range and nitpick comments (3)</summary><blockquote> <details> <summary>zetaclient/tss/tss_signer.go (3)</summary><blockquote> ...
codecov[bot] created a comment on a pull request on zeta-chain/node
## [Codecov](https://app.codecov.io/gh/zeta-chain/node/pull/3041?dropdown=coverage&src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=zeta-chai...
coderabbitai[bot] created a comment on a pull request on zeta-chain/node
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: skip review by coderabbit.ai --> > [!IMPORTANT] > ## Review skipped > > Draft detect...
brewmaster012 opened a draft pull request on zeta-chain/node
fix: replace DHT with private peer discovery
# Description <!--- Please include a summary of the changes and the related issue. Please also include relevant motivation and context. List any dependencies that are required for this change. -...
brewmaster012 pushed 1 commit to dht-removal zeta-chain/node
- replace DHT with authenticated discovery 84a4b29
gartnera created a review comment on a pull request on zeta-chain/node
```suggestion name: sim ``` I think it's good for the filename to match the workflow name
gartnera created a review comment on a pull request on zeta-chain/node
`main` branch isn't use. Remove please.
gartnera created a review comment on a pull request on zeta-chain/node
Please cleanup the formatting on this file
coderabbitai[bot] created a review comment on a pull request on zeta-chain/node
@kingpinXD, after reviewing the `unstake` operation, there doesn't appear to be any specific reason for `UnstakeMethodGas` to be set to 1000 while other staking operations are set to 10000. The `un...
kingpinXD created a review comment on a pull request on zeta-chain/node
Agree , we should check this value is there any reason why unstake would be less ?
kingpinXD created a review comment on a pull request on zeta-chain/node
Is it possible to check the `DistributeLog` ?
kingpinXD created a review comment on a pull request on zeta-chain/node
Can we check fee collector balance ?
kingpinXD created a review comment on a pull request on zeta-chain/node
What happens if this step fails ? Are the funds still locked or the whole tx is discarded ( like a regular cosmos msg )
kingpinXD created a review comment on a pull request on zeta-chain/node
How do we get this address ? some comments would be helpful