Skip to content
This repository has been archived by the owner on May 28, 2019. It is now read-only.

HDWallet support #14

Merged
merged 4 commits into from
Jul 11, 2018
Merged

HDWallet support #14

merged 4 commits into from
Jul 11, 2018

Conversation

alejandro-isaza
Copy link
Contributor

No description provided.

@alejandro-isaza alejandro-isaza changed the title HDWallet support HDWallet support WIP Jul 11, 2018
@alejandro-isaza alejandro-isaza changed the title HDWallet support WIP HDWallet support Jul 11, 2018
@codecov-io
Copy link

codecov-io commented Jul 11, 2018

Codecov Report

Merging #14 into master will decrease coverage by 1.03%.
The diff coverage is 77.88%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #14      +/-   ##
==========================================
- Coverage   85.39%   84.35%   -1.04%     
==========================================
  Files          39       40       +1     
  Lines        1554     1553       -1     
==========================================
- Hits         1327     1310      -17     
- Misses        227      243      +16
Impacted Files Coverage Δ
Sources/Data+Hex.swift 73.91% <ø> (ø)
Tests/PrivateKeyTests.swift 100% <100%> (ø)
Sources/Ethereum/Solidity/ABIEncoder.swift 89.71% <100%> (ø) ⬆️
Sources/Bitcoin/BitcoinAddress.swift 75.67% <100%> (ø) ⬆️
Sources/Ethereum/EthereumAddress.swift 90.24% <100%> (ø) ⬆️
Sources/Bitcoin/BitcoinPublicKey.swift 69.23% <100%> (ø) ⬆️
Sources/Ethereum/EthereumPublicKey.swift 69.23% <100%> (ø) ⬆️
Sources/Ethereum/Solidity/ENS/Namehash.swift 91.66% <100%> (ø) ⬆️
Sources/Ethereum/TransactionSigning.swift 94.87% <100%> (ø) ⬆️
Tests/Ethereum/EthereumAddressTests.swift 100% <100%> (ø) ⬆️
... and 15 more

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 98c16f1...35560fd. Read the comment docs.

public var defaultDerivationPath: DerivationPath {
switch self {
case .bitcoin:
return DerivationPath("m/44'/0'/0'/0/0")!
Copy link
Contributor

Choose a reason for hiding this comment

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

let's create class slips? so we can inject correct one in here

based on https://github.com/satoshilabs/slips/blob/a42019453c1d83bdcb51fca52434acce4a1b4ae2/slip-0044.md

@vikmeup vikmeup merged commit 9419678 into master Jul 11, 2018
@vikmeup vikmeup deleted the al/hdwallet branch July 11, 2018 04:31
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