Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add full version address identifies the hash_type #557

Merged
merged 5 commits into from
Sep 30, 2021

Conversation

Keith-CY
Copy link
Member

@Keith-CY Keith-CY commented Sep 1, 2021

Add a new format of full address, which identifies the hash_type

ref nervosnetwork/rfcs#239

@Keith-CY Keith-CY changed the title feat: extends hash_type to 'data' | 'type' | 'data1' feat: add full version address identified the hash_type and vm_version Sep 1, 2021
@Keith-CY Keith-CY marked this pull request as ready for review September 1, 2021 09:16
…_version

Add a new format of full address, which identifies the hash_type and vm_version

ref nervosnetwork/rfcs#239
@Keith-CY Keith-CY changed the title feat: add full version address identified the hash_type and vm_version feat: add full version address identifies the hash_type Sep 29, 2021
@Keith-CY Keith-CY force-pushed the add-new-address-format branch 2 times, most recently from d2e491d to 99ff9da Compare September 29, 2021 13:37
1. import bech32, bech32m from an external pacakge
2. update format of the new full address
@codecov-commenter
Copy link

codecov-commenter commented Sep 29, 2021

Codecov Report

Merging #557 (542d345) into extends-hash-type (00e7d87) will decrease coverage by 0.15%.
The diff coverage is 97.29%.

Impacted file tree graph

@@                  Coverage Diff                  @@
##           extends-hash-type     #557      +/-   ##
=====================================================
- Coverage              96.29%   96.14%   -0.16%     
=====================================================
  Files                     50       49       -1     
  Lines                   1566     1529      -37     
  Branches                 301      298       -3     
=====================================================
- Hits                    1508     1470      -38     
- Misses                    58       59       +1     
Impacted Files Coverage Δ
packages/ckb-sdk-utils/src/address/index.ts 98.33% <96.66%> (-1.67%) ⬇️
packages/ckb-sdk-utils/src/crypto/index.ts 100.00% <100.00%> (ø)
packages/ckb-sdk-utils/src/exceptions/address.ts 100.00% <100.00%> (ø)
packages/ckb-sdk-utils/src/index.ts 100.00% <100.00%> (ø)
packages/ckb-sdk-rpc/src/resultFormatter.ts 100.00% <0.00%> (ø)

@Keith-CY Keith-CY merged commit b304eb8 into extends-hash-type Sep 30, 2021
@Keith-CY Keith-CY deleted the add-new-address-format branch September 30, 2021 09:29
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.

None yet

3 participants