-
Notifications
You must be signed in to change notification settings - Fork 215
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Merged by Bors] - chore(txs): mempool optimization #6185
Conversation
bors try |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## develop #6185 +/- ##
=========================================
- Coverage 81.8% 81.8% -0.1%
=========================================
Files 301 301
Lines 32606 32605 -1
=========================================
- Hits 26697 26689 -8
- Misses 4178 4185 +7
Partials 1731 1731 ☔ View full report in Codecov by Sentry. |
func acceptable(err error) bool { | ||
return err == nil || errors.Is(err, errInsufficientBalance) || errors.Is(err, errTooManyNonce) | ||
return err == nil || errors.Is(err, errTooManyNonce) | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you explain in the PR description from a users perspective what this change will accomplish?
tryBuild failed: |
bors try |
tryBuild succeeded: |
bors merge |
## Motivation Improves some aspects of the mempool handling.
bors cancel |
Canceled. |
bors merge |
## Motivation Improves some aspects of the mempool handling.
Build failed:
|
bors merge |
## Motivation Improves some aspects of the mempool handling.
Pull request successfully merged into develop. Build succeeded: |
Motivation
Improves some aspects of the mempool handling.
Description
Test Plan
Tests passing
TODO