Skip to content
This repository has been archived by the owner on Nov 30, 2021. It is now read-only.

crypto: refactor for stargate #559

Merged
merged 10 commits into from
Oct 6, 2020
Merged

crypto: refactor for stargate #559

merged 10 commits into from
Oct 6, 2020

Conversation

fedekunze
Copy link
Contributor

No description provided.

@codecov
Copy link

codecov bot commented Oct 6, 2020

Codecov Report

Merging #559 into development will not change coverage.
The diff coverage is 93.33%.

Impacted file tree graph

@@             Coverage Diff              @@
##           development     #559   +/-   ##
============================================
  Coverage        70.71%   70.71%           
============================================
  Files               42       42           
  Lines             2339     2339           
============================================
  Hits              1654     1654           
  Misses             541      541           
  Partials           144      144           
Impacted Files Coverage Δ
crypto/ethsecp256k1/ethsecp256k1.go 65.51% <83.33%> (ø)
app/ante/ante.go 72.41% <100.00%> (ø)
crypto/ethsecp256k1/codec.go 100.00% <100.00%> (ø)
crypto/hd/algorithm.go 76.47% <100.00%> (ø)
x/evm/types/utils.go 41.50% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e7a19a1...ce4160d. Read the comment docs.

@fedekunze fedekunze marked this pull request as ready for review October 6, 2020 10:24
@fedekunze fedekunze requested a review from noot as a code owner October 6, 2020 10:24
@fedekunze fedekunze requested a review from araskachoi October 6, 2020 10:24
@fedekunze fedekunze merged commit 4e01da9 into development Oct 6, 2020
@fedekunze fedekunze deleted the crypto-refactor branch October 6, 2020 18:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants