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

fix: convert output of multihash.decode to buffer #71

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

achingbrain
Copy link
Member

The multihash module returns Uint8Arrays from it's decode method but
smart-buffer requires everything to be a buffer so convert the output.

The multihash module returns Uint8Arrays from it's decode method but
smart-buffer requires everything to be a buffer so convert the output.
@achingbrain achingbrain requested a review from vmx August 5, 2020 10:06
@codecov
Copy link

codecov bot commented Aug 5, 2020

Codecov Report

Merging #71 into master will increase coverage by 0.05%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #71      +/-   ##
==========================================
+ Coverage   84.90%   84.96%   +0.05%     
==========================================
  Files           8        8              
  Lines         265      266       +1     
==========================================
+ Hits          225      226       +1     
  Misses         40       40              
Impacted Files Coverage Δ
src/util.js 90.90% <100.00%> (ø)
src/util/util.js 92.30% <100.00%> (+0.20%) ⬆️

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 989e20d...65a3042. Read the comment docs.

@vmx vmx merged commit 20dd68f into master Aug 5, 2020
@vmx vmx deleted the fix/convert-uint8array-to-buffer branch August 5, 2020 10:17
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.

2 participants