-
Notifications
You must be signed in to change notification settings - Fork 261
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: introduce FirstCompactShareContentSize
#804
feat: introduce FirstCompactShareContentSize
#804
Conversation
Part of celestiaorg#725 Motivation: we will soon introduce a new appconst for `FirstCompactShareContentSize`
Codecov Report
@@ Coverage Diff @@
## main #804 +/- ##
=======================================
Coverage 23.31% 23.31%
=======================================
Files 71 71
Lines 8792 8792
=======================================
Hits 2050 2050
Misses 6570 6570
Partials 172 172 Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
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.
disabled auto merge to provide time to pursue another option if you want as it sounded like that wasn't an ideal solution, but I am satisfied, so I'm approving
🙏
This reverts commit 285d1a2.
as noticed in #804 (comment), I forgot to delete the old `DelimLen` function after fixing a bug in #747.
~~Blocked on celestiaorg#803 Part of celestiaorg#725
as noticed in celestiaorg#804 (comment), I forgot to delete the old `DelimLen` function after fixing a bug in celestiaorg#747.
Blocked on #803Part of #725