-
Notifications
You must be signed in to change notification settings - Fork 129
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
chore(proto): fixes and CI check #2992
Conversation
10afac6
to
528315b
Compare
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## development #2992 +/- ##
===============================================
- Coverage 52.08% 51.81% -0.28%
===============================================
Files 220 220
Lines 27716 27716
===============================================
- Hits 14436 14360 -76
- Misses 12043 12125 +82
+ Partials 1237 1231 -6 |
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.
Nice!
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.
lgtm!
🎉 This PR is included in version 0.8.0 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Changes
make proto
was not working, found whilst reviewing feat(dot/sync): Remove theEndBlockHash
fromBlockRequestMessage
#2977go generate
to generate Go code from proto fileprotoc
requirement when runningmake proto
Tests
Issues
Primary Reviewer
@EclesioMeloJunior