-
Notifications
You must be signed in to change notification settings - Fork 100
New issue
Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? # to your account
imp: re-export Timestamp
& Duration
from ibc_proto::google::protobuf
#1054
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1054 +/- ##
=======================================
Coverage 71.30% 71.30%
=======================================
Files 187 187
Lines 18507 18507
=======================================
Hits 13196 13196
Misses 5311 5311 ☔ View full report in Codecov by Sentry. |
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.
Asked a question to confirm if we need to re-export everything.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #1054 +/- ##
==========================================
- Coverage 67.05% 67.04% -0.02%
==========================================
Files 204 205 +1
Lines 20552 20567 +15
==========================================
+ Hits 13782 13789 +7
- Misses 6770 6778 +8 ☔ View full report in Codecov by Sentry. |
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.
Great ! Thanks 🙏
ibc_proto::google::protobuf
Timestamp
& Duration
from ibc_proto::google::protobuf
Signed-off-by: Farhad Shabani <Farhad.Shabani@gmail.com>
…obuf` (#1054) * imp: re-export all the proto type under ibc_proto::google::protobuf * fix: re-export a few necessary types --------- Signed-off-by: Farhad Shabani <Farhad.Shabani@gmail.com>
Description
This requirement emerged while implementing the
sov-celestia
light client, eliminating the need to import the same types fromtendermint_proto::google::protobuf
.PR author checklist:
unclog
.docs/
).