Skip to content

Refactor: Header -> Signed Header#771

Merged
nashqueue merged 9 commits intomainfrom
signed-header
Mar 15, 2023
Merged

Refactor: Header -> Signed Header#771
nashqueue merged 9 commits intomainfrom
signed-header

Conversation

@nashqueue
Copy link
Copy Markdown
Contributor

@nashqueue nashqueue commented Mar 13, 2023

Fixes: #773

@nashqueue nashqueue self-assigned this Mar 13, 2023
@nashqueue nashqueue added this to the Go-header implementation milestone Mar 13, 2023
@gupadhyaya gupadhyaya force-pushed the signed-header branch 2 times, most recently from d93696f to 53a82fe Compare March 13, 2023 22:45
@codecov-commenter
Copy link
Copy Markdown

Codecov Report

Patch coverage: 62.50% and project coverage change: -0.18 ⚠️

Comparison is base (8eb79aa) 55.15% compared to head (f29c030) 54.97%.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #771      +/-   ##
==========================================
- Coverage   55.15%   54.97%   -0.18%     
==========================================
  Files          53       53              
  Lines       10000     9917      -83     
==========================================
- Hits         5515     5452      -63     
+ Misses       3660     3656       -4     
+ Partials      825      809      -16     
Impacted Files Coverage Δ
types/pb/rollkit/rollkit.pb.go 38.77% <21.31%> (-1.62%) ⬇️
state/executor.go 64.90% <86.48%> (+0.16%) ⬆️
block/manager.go 63.96% <94.11%> (+0.24%) ⬆️
conv/abci/block.go 94.33% <100.00%> (ø)
da/mock/mock.go 73.75% <100.00%> (ø)
node/full_client.go 54.65% <100.00%> (ø)
store/store.go 60.89% <100.00%> (ø)
types/hashing.go 100.00% <100.00%> (ø)
types/serialization.go 77.96% <100.00%> (+0.78%) ⬆️
types/validation.go 37.20% <100.00%> (ø)

... and 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Comment thread proto/rollkit/rollkit.proto
Comment thread proto/rollkit/rollkit.proto
@gupadhyaya gupadhyaya self-requested a review March 15, 2023 16:48
gupadhyaya
gupadhyaya previously approved these changes Mar 15, 2023
tuxcanfly
tuxcanfly previously approved these changes Mar 15, 2023
Copy link
Copy Markdown
Collaborator

@tuxcanfly tuxcanfly left a comment

Choose a reason for hiding this comment

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

Some nits, non blocking.

Comment thread proto/rollkit/rollkit.proto Outdated
Comment thread proto/rollkit/rollkit.proto Outdated
Comment thread types/block.go
Manav-Aggarwal
Manav-Aggarwal previously approved these changes Mar 15, 2023
@nashqueue nashqueue dismissed stale reviews from Manav-Aggarwal and tuxcanfly via 172372e March 15, 2023 17:19
@nashqueue nashqueue merged commit 4039cbe into main Mar 15, 2023
@nashqueue nashqueue deleted the signed-header branch March 15, 2023 17:33
nashqueue added a commit that referenced this pull request Apr 10, 2023
Fixes: #773

---------

Co-authored-by: Ganesha Upadhyaya <gupadhyaya@Ganeshas-MacBook-Pro-2.local>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Refactor (Header, Last Commit) into Signed Header

5 participants