dymensionxyz / dymint

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

feat(p2p): refactor p2p config options #861

Closed srene closed 1 month ago

srene commented 1 month ago

PR Standards

All p2p config parameters have been moved to P2P to specific paramaters, to avoid confusion with Tendermint p2p options

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


Close #797

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

For Author:


For Reviewer:


After reviewer approval:

codecov[bot] commented 1 month ago

Codecov Report

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

Project coverage is 39.65%. Comparing base (851b312) to head (1913a9e). Report is 25 commits behind head on main.

:exclamation: Current head 1913a9e differs from pull request most recent head 842a7b7. Consider uploading reports for the commit 842a7b7 to get more accurate results

Files Patch % Lines
config/flags.go 33.33% 8 Missing and 4 partials :warning:
config/p2p.go 25.00% 4 Missing and 2 partials :warning:
config/config.go 0.00% 2 Missing and 1 partial :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #861 +/- ## ========================================== - Coverage 40.44% 39.65% -0.80% ========================================== Files 120 118 -2 Lines 18935 18703 -232 ========================================== - Hits 7659 7416 -243 - Misses 10201 10232 +31 + Partials 1075 1055 -20 ```

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