Skip to content
This repository has been archived by the owner on Dec 6, 2022. It is now read-only.

fix: remove redundant multihashes dep #66

Merged
merged 1 commit into from
Aug 5, 2020

Conversation

achingbrain
Copy link
Member

The multihashes module is bundled with multihashing-async so no need to depend on it as well.

The multihashes module is bundled with multihashing-async so no
need to depend on it as well.
@achingbrain achingbrain requested a review from vmx August 5, 2020 10:12
@codecov
Copy link

codecov bot commented Aug 5, 2020

Codecov Report

Merging #66 into master will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##            master       #66   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            4         4           
  Lines           69        69           
=========================================
  Hits            69        69           
Impacted Files Coverage Δ
src/util.js 100.00% <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 186efda...a61f836. Read the comment docs.

@vmx vmx merged commit 40a4221 into master Aug 5, 2020
@vmx vmx deleted the fix/remove-redundant-multihashes-dep branch August 5, 2020 12:56
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