Skip to content

chore: update @arkecosystem/utils dependency#3665

Merged
faustbrian merged 2 commits intomasterfrom
update-deps
Apr 28, 2020
Merged

chore: update @arkecosystem/utils dependency#3665
faustbrian merged 2 commits intomasterfrom
update-deps

Conversation

@faustbrian
Copy link
Copy Markdown
Contributor

@faustbrian faustbrian commented Apr 28, 2020

A missing dependency in an older release of @arkecosystem/utils causes issues when used with yarn berry.

    @arkecosystem/utils tried to access deepmerge, but it isn't declared in its dependencies; this makes the require call ambiguous and unsound.

    Required package: deepmerge (via "deepmerge")
    Required by: @arkecosystem/utils@npm:0.8.3 (via /Users/brian/Code/platform-sdk/.yarn/cache/@arkecosystem-utils-npm-0.8.3-595b56811a-2.zip/node_modules/@arkecosystem/utils/dist/)

@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 28, 2020

Codecov Report

Merging #3665 into master will increase coverage by 0.00%.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #3665   +/-   ##
=======================================
  Coverage   66.06%   66.07%           
=======================================
  Files         441      441           
  Lines       12513    12513           
  Branches     1717     1717           
=======================================
+ Hits         8267     8268    +1     
+ Misses       4213     4212    -1     
  Partials       33       33           
Impacted Files Coverage Δ
packages/core-transaction-pool/src/storage.ts 100.00% <0.00%> (+2.50%) ⬆️

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 5911c7c...5eac4aa. Read the comment docs.

@faustbrian faustbrian merged commit 76283d3 into master Apr 28, 2020
@ghost ghost deleted the update-deps branch April 28, 2020 06:01
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.

1 participant