Skip to content

Fix tx DecodeRLP for rlp lists #1801

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
merged 2 commits into from
Jan 24, 2022
Merged

Conversation

gastonponti
Copy link
Contributor

Description

Fix transaction decode

@gastonponti gastonponti requested a review from a team as a code owner January 20, 2022 22:17
@gastonponti gastonponti requested review from hbandura and nategraf and removed request for a team January 20, 2022 22:17
@piersy
Copy link
Contributor

piersy commented Jan 20, 2022

Coverage from tests in ./e2e_test/... for ./consensus/istanbul/... at commit c858f82

coverage: 50.5% of statements across all listed packages
coverage:  65.9% of statements in consensus/istanbul
coverage:  39.8% of statements in consensus/istanbul/announce
coverage:  54.4% of statements in consensus/istanbul/backend
coverage:   0.0% of statements in consensus/istanbul/backend/backendtest
coverage:  24.3% of statements in consensus/istanbul/backend/internal/replica
coverage:  69.5% of statements in consensus/istanbul/core
coverage:  45.0% of statements in consensus/istanbul/db
coverage:   0.0% of statements in consensus/istanbul/proxy
coverage:  75.3% of statements in consensus/istanbul/uptime
coverage: 100.0% of statements in consensus/istanbul/uptime/store
coverage:  51.8% of statements in consensus/istanbul/validator
coverage:  79.2% of statements in consensus/istanbul/validator/random
CommentID: 284977434a

Copy link

@nategraf nategraf left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Spent some time looking into this, and this fix looks good 👍

@gastonponti gastonponti merged commit f957b0a into master Jan 24, 2022
@gastonponti gastonponti deleted the gastonponti/fix-decode-rlp branch January 24, 2022 21:49
@piersy
Copy link
Contributor

piersy commented Jan 25, 2022

@gastonponti what did this PR fix?

gastonponti added a commit that referenced this pull request Jan 25, 2022
gastonponti added a commit that referenced this pull request Jan 25, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants