Closed golangisfun123 closed 2 weeks ago
[!WARNING]
Rate limit exceeded
@golangisfun123 has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 53 minutes and 45 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 1ec0562b1e7da6596df7c463160b9c65119cfefd and 888792fb80aa71ecb8aa064ffc12ec928d5ae7d5.
The updates to the ethergo
package introduce enhanced block fetching with finality modes and block wait parameters in the chainListener
struct. Changes include new options for configuring finality mode and block wait, restructured block number retrieval for better compatibility, and refactored test cases to validate the new functionalities.
File | Change Summary |
---|---|
ethergo/listener/listener.go |
Added finalityMode and blockWait to chainListener , updated NewChainListener , replaced block fetch calls, and added getBlockNumber method. |
ethergo/listener/listener_test.go |
Reordered imports, updated TestListenForEvents function, refactored NewChainListener calls, added event handling, and block comparisons. |
ethergo/listener/options.go |
Introduced WithFinalityMode and WithBlockWait functions to set finality mode and block wait options in chainListener . |
sequenceDiagram
participant Client
participant ChainListener
Client->>ChainListener: NewChainListener(opts...)
ChainListener->>ChainListener: set finalityMode, blockWait
Note right of ChainListener: Initialize chainListener with options
Client->>ChainListener: Start listening for events
ChainListener->>ChainListener: doPoll method
ChainListener->>ChainListener: getBlockNumber()
Note right of ChainListener: Retrieve block number considering finalityMode and blockWait
Changes blown by winds of code, In the chains where blocks erode, Options bloom, new paths are shown, Finality waits, our seeds are sown. With each poll, we fetch and see, The future of our blockchain be.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Attention: Patch coverage is 70.37037%
with 8 lines
in your changes missing coverage. Please review.
Project coverage is 26.20226%. Comparing base (
3eccadb
) to head (888792f
). Report is 2 commits behind head on master.
Files | Patch % | Lines |
---|---|---|
ethergo/listener/listener.go | 57.14286% | 4 Missing and 2 partials :warning: |
ethergo/listener/options.go | 84.61538% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Latest commit: |
888792f
|
Status: | ✅ Deploy successful! |
Preview URL: | https://654cf5d5.sanguine-fe.pages.dev |
Branch Preview URL: | https://feat-listener-finality.sanguine-fe.pages.dev |
Description A clear and concise description of the features you're adding in this pull request.
Additional context Add any other context about the problem you're solving.
Metadata
Summary by CodeRabbit
New Features
Bug Fixes