-
Notifications
You must be signed in to change notification settings - Fork 3.6k
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
R4R: AnteHandler Cleanup #2950
R4R: AnteHandler Cleanup #2950
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #2950 +/- ##
========================================
Coverage 55.53% 55.53%
========================================
Files 120 120
Lines 8494 8494
========================================
Hits 4717 4717
Misses 3455 3455
Partials 322 322 |
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.
All looks clean and tidy. Good to go
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.
If we end up removing the account number / sequence number per #2952, this newly added error code is obsolete. (And is thus creating an unnecessary future breaking change) I propose instead just using ErrInternal
Updated @ValarDragon to use |
@alexanderbez Looks like this just needs a minor rebase? Glad to quickly once-over again after, then should be OK to merge. |
Yes, I'll fix asap. |
@cwgoes updated. I no longer think this warrants a pending log update. |
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.
Tested ACK (no substantive changes - but great cleanup!)
* Add and use CodeInvalidAccountNumber in ante handler * Minor cleanup of ante handler * Add pending log entry * Spell check and more minor comment restructuring * Add additional CodeInvalidAccountNumber * Update validateAccNumAndSequence * Revert account number error in favor of CodeInternal * Update pending log * Fix pending log
CodeInternal
for invalid account numbersTargeted PR against correct branch (see CONTRIBUTING.md)
Linked to github-issue with discussion and accepted design OR link to spec that describes this work.
Wrote tests
Updated relevant documentation (
docs/
)Added entries in
PENDING.md
with issue #rereviewed
Files changed
in the github PR explorerFor Admin Use: