Codecov Report

Merging #329 (6dce0c8) into dev (6add1ea) will not change coverage.
The diff coverage is n/a.

❗ Current head 6dce0c8 differs from pull request most recent head 685c8f2. Consider uploading reports for the commit 685c8f2 to get more accurate results

@@           Coverage Diff           @@
##              dev     #329   +/-   ##
=======================================
  Coverage   79.19%   79.19%           
=======================================
  Files          45       45           
  Lines        1817     1817           
  Branches      304      304           
=======================================
  Hits         1439     1439           
  Misses        377      377           
  Partials        1        1           

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 6add1ea...685c8f2. Read the comment docs.

This seems like a breaking change by removing the zilliqa-js/viewblock package from the library? The package @zilliqa-js/viewblock is developed by Viewblock to allow developers to access viewblock API via Zilliqa-JS SDK.

If we planned to remove the package, I think we can update our main readme to ask users to run yarn add @zilliqa-js/viewblock as per https://www.npmjs.com/package/@zilliqa-js/viewblock if they need to access viewblock?

Edit: empty package doesn't do anything. Users still need to perform yarn add @zilliqa-js/viewblock if they need to access viewblock api. Safe to remove unless empty package is used to raise awareness of viewblock API?