Skip to content

remove unnecessary "crypto" import alias#2940

Merged
melekes merged 1 commit intodevelopfrom
dev/remove_alias
Nov 28, 2018
Merged

remove unnecessary "crypto" import alias#2940
melekes merged 1 commit intodevelopfrom
dev/remove_alias

Conversation

@ValarDragon
Copy link
Contributor

This is non-breaking, we just missed a few spots when previously merging the repos.

  • Updated all relevant documentation in docs - n/a
  • Updated all code comments where relevant - n/a
  • Wrote tests - n/a
  • Updated CHANGELOG_PENDING.md - n/a, not outward facing change

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (develop@3f987ad). Click here to learn what that means.
The diff coverage is n/a.

@@            Coverage Diff            @@
##             develop   #2940   +/-   ##
=========================================
  Coverage           ?   62.7%           
=========================================
  Files              ?     212           
  Lines              ?   17339           
  Branches           ?       0           
=========================================
  Hits               ?   10873           
  Misses             ?    5551           
  Partials           ?     915
Impacted Files Coverage Δ
p2p/key.go 70.21% <ø> (ø)
p2p/pex/addrbook.go 68.91% <ø> (ø)
p2p/test_util.go 66.41% <ø> (ø)
tools/tm-monitor/monitor/node.go 60.34% <ø> (ø)
rpc/core/pipe.go 30.23% <ø> (ø)

@melekes melekes merged commit 4039276 into develop Nov 28, 2018
@melekes melekes deleted the dev/remove_alias branch November 28, 2018 19: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