Skip to content
This repository was archived by the owner on Dec 5, 2021. It is now read-only.

Comments

use StaticJsonRpcProvider where possible#461

Merged
CAPtheorem merged 4 commits intodevelopfrom
reduce-rpc-calls
Sep 15, 2021
Merged

use StaticJsonRpcProvider where possible#461
CAPtheorem merged 4 commits intodevelopfrom
reduce-rpc-calls

Conversation

@CAPtheorem
Copy link

Change to StaticJsonRpcProvider where possible

@codecov-commenter
Copy link

codecov-commenter commented Sep 15, 2021

Codecov Report

Merging #461 (bfa0bac) into develop (25c5bbc) will not change coverage.
The diff coverage is 0.00%.

❗ Current head bfa0bac differs from pull request most recent head 5e017b4. Consider uploading reports for the commit 5e017b4 to get more accurate results
Impacted file tree graph

@@           Coverage Diff            @@
##           develop     #461   +/-   ##
========================================
  Coverage    76.09%   76.09%           
========================================
  Files           81       81           
  Lines         3037     3037           
  Branches       469      469           
========================================
  Hits          2311     2311           
  Misses         726      726           
Flag Coverage Δ
batch-submitter 59.51% <ø> (ø)
contracts 86.05% <ø> (ø)
core-utils 68.16% <ø> (ø)
data-transport-layer 37.68% <ø> (ø)
message-relayer 72.22% <0.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
packages/message-relayer/src/relay-tx.ts 71.15% <0.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 b91354b...5e017b4. Read the comment docs.

@CAPtheorem CAPtheorem merged commit cd660f5 into develop Sep 15, 2021
@CAPtheorem CAPtheorem deleted the reduce-rpc-calls branch September 15, 2021 22:28
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants