celestia-node icon indicating copy to clipboard operation
celestia-node copied to clipboard

fix: interface compliance check for `API` struct

Open brawncode opened this issue 5 months ago • 1 comments

updated the code to properly verify that API implements the da.DA interface. using var _ da.DA = (*API)(nil) ensures compile-time safety without any runtime overhead. all interface methods are correctly delegated to Internal, so this now reflects the intended design.

brawncode avatar Aug 14 '25 11:08 brawncode

Codecov Report

:white_check_mark: All modified and coverable lines are covered by tests. :white_check_mark: Project coverage is 35.41%. Comparing base (2469e7a) to head (1e3a2b7). :warning: Report is 582 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #4488      +/-   ##
==========================================
- Coverage   44.83%   35.41%   -9.42%     
==========================================
  Files         265      304      +39     
  Lines       14620    24298    +9678     
==========================================
+ Hits         6555     8606    +2051     
- Misses       7313    14745    +7432     
- Partials      752      947     +195     

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

:rocket: New features to boost your workflow:
  • :snowflake: Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

codecov-commenter avatar Sep 16 '25 13:09 codecov-commenter