-
Notifications
You must be signed in to change notification settings - Fork 144
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
feat: Pull protos from services #2611
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: ivaylogarnev-limechain <[email protected]>
Signed-off-by: ivaylogarnev-limechain <[email protected]>
Signed-off-by: ivaylogarnev-limechain <[email protected]>
Signed-off-by: ivaylogarnev-limechain <[email protected]>
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2611 +/- ##
==========================================
+ Coverage 84.71% 84.84% +0.13%
==========================================
Files 283 284 +1
Lines 71086 71309 +223
==========================================
+ Hits 60218 60503 +285
+ Misses 10868 10806 -62 ☔ View full report in Codecov by Sentry. |
Signed-off-by: ivaylogarnev-limechain <[email protected]>
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.
Looks good
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
…gnore Signed-off-by: ivaylogarnev-limechain <[email protected]>
2fdbe34
Signed-off-by: ivaylogarnev-limechain <[email protected]>
Quality Gate passedIssues Measures |
Description:
We need to migrate the protobuf definitions from the hedera-protobufs repository to pull from the hedera-services repository. This change is aimed at simplifying the development process and centralizing all Protobuf-related files directly within the hedera-services repo, ensuring they stay in sync with service code.
Related issue(s):
Fixes #
#2572
Checklist