Skip to content

refactor(init): Remove duplicate code and simplify full verification method#662

Merged
msbrogli merged 1 commit intomasterfrom
refactor/init-full-verification
Jun 14, 2023
Merged

refactor(init): Remove duplicate code and simplify full verification method#662
msbrogli merged 1 commit intomasterfrom
refactor/init-full-verification

Conversation

@msbrogli
Copy link
Member

@msbrogli msbrogli commented Jun 14, 2023

Acceptance criteria

  1. Rename HathorManager._initialize_components() to _initialize_components_full_verification().
  2. Create method HathorManager._verify_soft_voided_txs().
  3. Use HathorManager._sync_v2_resume_validations() on _initialize_components_full_verification.

@msbrogli msbrogli self-assigned this Jun 14, 2023
@msbrogli msbrogli requested a review from jansegre as a code owner June 14, 2023 06:14
@msbrogli msbrogli requested a review from glevco June 14, 2023 06:16
@codecov
Copy link

codecov bot commented Jun 14, 2023

Codecov Report

Merging #662 (ea7b859) into master (33daa68) will increase coverage by 0.14%.
The diff coverage is 72.22%.

@@            Coverage Diff             @@
##           master     #662      +/-   ##
==========================================
+ Coverage   83.63%   83.78%   +0.14%     
==========================================
  Files         246      246              
  Lines       20339    20309      -30     
  Branches     2767     2753      -14     
==========================================
+ Hits        17011    17016       +5     
+ Misses       2742     2711      -31     
+ Partials      586      582       -4     
Impacted Files Coverage Δ
hathor/manager.py 74.19% <72.22%> (+4.41%) ⬆️

... and 4 files with indirect coverage changes

@msbrogli msbrogli force-pushed the refactor/init-full-verification branch from ea7b859 to f772aac Compare June 14, 2023 14:02
@msbrogli msbrogli merged commit f772aac into master Jun 14, 2023
@msbrogli msbrogli deleted the refactor/init-full-verification branch June 14, 2023 14:03
@jansegre jansegre mentioned this pull request Jul 12, 2023
2 tasks
This was referenced Aug 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants