Ecosyste.ms: Timeline
Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:hammer_and_wrench: Refactor suggestion_ **Add documentation and improve channel operation safety.** The function needs documentation and could benefit from non-blocking channel operations to pr...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:hammer_and_wrench: Refactor suggestion_ **Improve context and error handling.** The function has several areas that could be enhanced: 1. Consider using a parent context instead of creating a n...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:bulb: Codebase verification_ **Validation for ConsensusNodeThreshold is needed** Based on the code analysis, the threshold value is used directly in the tracker without any validation. The trac...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Address documentation and validation concerns in GrpcNodeConfig.** 1. The GRPCURL field comment incorrectly refers to "GRPCURLs" (plural) instead of "GRPCURL" 2. Con...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Improve error handling and optimize threshold comparison.** The implementation needs several improvements: 1. Error messages should be consistent in format 2. Status...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Add input validation to constructor.** The constructor should validate that the threshold is greater than 0 to prevent invalid states. Apply this diff: ```diff +//...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Improve struct definitions and documentation.** The current implementation has several limitations: 1. Using `uint8` limits threshold to 255, which might be too rest...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Fix constant declarations and add documentation.** Several issues need to be addressed: 1. Typo in `statusProcessig` (should be `statusProcessing`) 2. Incorrect usag...
backsapc pushed 16 commits to feature/683-keychain-sdk-configure-multiple-nodes-endpoints warden-protocol/wardenprotocol
- docs: fix logo path in README 5b6dc57
- 822 Act & Warden Precompiles (#946) * precompile boilerplate * boileplate for act precompile * rename warden.g... 6b55cb7
- Prepare release v0.5.2 e9ba0b8
- build(deps): bump the npm_and_yarn group across 2 directories with 1 update Bumps the npm_and_yarn group with 1 upda... dd20e71
- build(deps): bump the go_modules group across 2 directories with 2 updates Bumps the go_modules group with 2 updates... 4f28e5c
- fix(spaceward): fix for maintenance mode issue in chiado (#950) 48e4d05
- added chiado updates (#890) * added chiado updates * coderabbit * improved announcements * replaced comet w... f49919a
- fix(spaceward): debugging maintenance mode issue (#952) 49a2e18
- fix(spaceward): debugging changes reverted (#953) 7e70899
- fix(spaceward): update to maintenance mode check (#956) 035a0fa
- fix(docs): update fees and gas amounts to join Chiado (#960) this will change the amounts to be more reasonable cb607e9
- fix(spaceward): another maintenance issue fix (#961) 52e2ec4
- Docs evm contract fixes (#901) * adjustments in the evm guide * changes based on testing the guide with v0.5.1 ... 4440893
- adjustments in the wasm guide (#902) * adjustments in the wasm guide * a small fix * changes based on testing ... a4a0eb1
- added titles to code snippets (#962) ace2aa9
- Merge branch 'main' into feature/683-keychain-sdk-configure-multiple-nodes-endpoints 50fffb0
backsapc pushed 1 commit to feature/683-keychain-sdk-configure-multiple-nodes-endpoints warden-protocol/wardenprotocol
- Updated config for wardenkms Lint fixes Merge conflicts Added ClientsPool abstraction Build fixes Lint fix Lint... cd821e6
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Add struct documentation and threshold validation.** The `T` struct lacks documentation explaining its purpose. Additionally, the `threshold` field should be validat...
backsapc created a review comment on a pull request on warden-protocol/wardenprotocol
@Pitasi never mind, @artur-abliazimov helped me realize it's not an issue, so I kept the deletion from the tracker as is.
backsapc created a review comment on a pull request on warden-protocol/wardenprotocol
You're right, I fixed it. The problem is that if we keep removing requests from the tracker when they’re fulfilled, we risk executing them more than once due to possible stale nodes. So, we need to...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:hammer_and_wrench: Refactor suggestion_ **Avoid hardcoding chain and contract addresses.** Hardcoding destination chain and contract address reduces flexibility and maintainability. Consider m...
coderabbitai[bot] created a review on a pull request on warden-protocol/wardenprotocol
**Actionable comments posted: 1** <details> <summary>🧹 Outside diff range and nitpick comments (5)</summary><blockquote> <details> <summary>docs/developer-docs/docs/build-an-app/deploy-smart-cont...
coderabbitai[bot] created a review on a pull request on warden-protocol/wardenprotocol
**Actionable comments posted: 1** <details> <summary>🧹 Outside diff range and nitpick comments (5)</summary><blockquote> <details> <summary>docs/developer-docs/docs/build-an-app/deploy-smart-cont...
vercel[bot] created a comment on a pull request on warden-protocol/wardenprotocol
[vc]: #Cz6Zq/APjR4caH9RLI7nclM3vIzgbJNWDxOLwHSyLy4=:eyJpc01vbm9yZXBvIjp0cnVlLCJ0eXBlIjoiZ2l0aHViIiwicHJvamVjdHMiOlt7Im5hbWUiOiJ3YXJkZW4taGVscC1jZW50ZXIiLCJyb290RGlyZWN0b3J5IjoiZG9jcy9oZWxwLWNlbnRlc...
alijnmerchant21 created a review on a pull request on warden-protocol/wardenprotocol
Assuming no changes made to code. Thanks!
coderabbitai[bot] created a comment on a pull request on warden-protocol/wardenprotocol
<!-- This is an auto-generated comment: summarize by coderabbit.ai --> <!-- This is an auto-generated comment: rate limited by coderabbit.ai --> > [!WARNING] > ## Rate limit exceeded > > @ijonele...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:hammer_and_wrench: Refactor suggestion_ **Add documentation and improve channel handling.** The function needs documentation and could benefit from non-blocking channel operations. Add documen...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:hammer_and_wrench: Refactor suggestion_ **Consider improving error resilience and configuration.** The function has several areas that could be enhanced: 1. The hardcoded 5-second timeout shou...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:hammer_and_wrench: Refactor suggestion_ **Improve error messages and optimize threshold comparison.** The implementation could be enhanced for better clarity and efficiency: 1. Error messages s...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Consider using uint instead of uint8 for threshold.** The uint8 type limits the threshold to a maximum value of 255, which might be too restrictive for future use ca...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Fix potential race condition in statusTracker method.** The method modifies the `ingested` map without holding the write lock, which could lead to race conditions. C...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Fix lock ordering in Ingest method.** The method holds a read lock while calling `statusTracker` which acquires a write lock. This could lead to deadlocks. Consider ...
coderabbitai[bot] created a review comment on a pull request on warden-protocol/wardenprotocol
_:warning: Potential issue_ **Handle error from MarkProcessed.** The error from `MarkProcessed` is silently ignored. Consider either logging the error or returning it to the caller for proper err...