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

Upgrade to ethereumjs-config v2 #289

Merged
merged 3 commits into from
Feb 5, 2021
Merged

Upgrade to ethereumjs-config v2 #289

merged 3 commits into from
Feb 5, 2021

Conversation

ryanio
Copy link
Contributor

@ryanio ryanio commented Feb 4, 2021

This PR upgrades this repository to ethereumjs-config v2, switches from coveralls to codecov, and fixes new lint errors. A few functions are moved up in their respective files to satisfy the no-use-before-define rule.

Closes #286.

* switch from coveralls to codecov
* fix lint errors
  * add tsconfig to silence errors
  * remove custom types exclude (seems to be working without now)
  * lint
@codecov
Copy link

codecov bot commented Feb 4, 2021

Codecov Report

❗ No coverage uploaded for pull request base (master@0725f7e). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #289   +/-   ##
=========================================
  Coverage          ?   97.10%           
=========================================
  Files             ?       11           
  Lines             ?      415           
  Branches          ?       71           
=========================================
  Hits              ?      403           
  Misses            ?        3           
  Partials          ?        9           

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 0725f7e...919ff36. Read the comment docs.

Copy link
Member

@holgerd77 holgerd77 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, great, thanks for the PR, nice to have this updated now! 😄

@holgerd77 holgerd77 merged commit fe518cf into master Feb 5, 2021
@holgerd77 holgerd77 deleted the upgrade-config-2.0 branch February 5, 2021 11:09
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.

Upgrade ethereumjs-config to v2, switch from Coveralls to CodeCov
2 participants