dymensionxyz / dymint

Sequencing Engine for Dymension RollApps
Other
90 stars 61 forks source link

fix(manager): removes outdated syncTarget field #854

Closed danwt closed 1 month ago

danwt commented 1 month ago

PR Standards

Opening a pull request should be able to meet the following requirements


Close https://github.com/dymensionxyz/dymint/issues/850

<-- Briefly describe the content of this pull request -->

For Author:


For Reviewer:


After reviewer approval:

omritoptix commented 1 month ago

@danwt conflicts

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 80.48780% with 8 lines in your changes are missing coverage. Please review.

Project coverage is 39.14%. Comparing base (851b312) to head (448db03). Report is 33 commits behind head on main.

:exclamation: Current head 448db03 differs from pull request most recent head 74ce466. Consider uploading reports for the commit 74ce466 to get more accurate results

Files Patch % Lines
block/synctarget.go 11.11% 8 Missing :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #854 +/- ## ========================================== - Coverage 40.44% 39.14% -1.31% ========================================== Files 120 120 Lines 18935 18602 -333 ========================================== - Hits 7659 7282 -377 - Misses 10201 10281 +80 + Partials 1075 1039 -36 ```

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