Skip to content

Remove TTD of params#315

Merged
AntiD2ta merged 11 commits intodevelopfrom
fix/geth-ttd-patch
Oct 13, 2023
Merged

Remove TTD of params#315
AntiD2ta merged 11 commits intodevelopfrom
fix/geth-ttd-patch

Conversation

@stdevMac
Copy link
Copy Markdown
Member

@stdevMac stdevMac commented Oct 3, 2023

Changes:

  • Remove TTD of the params, including test cases, command flags and geth patch for custom testnets.

Types of changes

  • Bugfix (non-breaking change which fixes an issue)

@stdevMac stdevMac self-assigned this Oct 3, 2023
@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Oct 3, 2023

Codecov Report

All modified lines are covered by tests ✅

Comparison is base (e6b68c7) 86.75% compared to head (2652cf2) 86.74%.

Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #315      +/-   ##
===========================================
- Coverage    86.75%   86.74%   -0.02%     
===========================================
  Files           78       78              
  Lines         4582     4548      -34     
===========================================
- Hits          3975     3945      -30     
+ Misses         522      518       -4     
  Partials        85       85              
Flag Coverage Δ
unittests 86.74% <100.00%> (-0.02%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
cli/cli.go 68.75% <ø> (+0.88%) ⬆️
cli/generate.go 90.37% <ø> (-0.17%) ⬇️
cli/sub_gen.go 93.68% <100.00%> (-2.07%) ⬇️
configs/init.go 100.00% <ø> (ø)
internal/pkg/env/get_variable.go 100.00% <ø> (ø)
internal/pkg/generate/generate_scripts.go 96.51% <ø> (-0.05%) ⬇️
internal/pkg/generate/types.go 100.00% <ø> (ø)

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Copy link
Copy Markdown
Collaborator

@AntiD2ta AntiD2ta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please update the Changelog, the rest LGTM

@AntiD2ta AntiD2ta merged commit 55d554d into develop Oct 13, 2023
@AntiD2ta AntiD2ta deleted the fix/geth-ttd-patch branch October 13, 2023 05:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants