Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

chore: remove unnecessary EVM client interface #109

Merged
merged 2 commits into from
Feb 9, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Feb 9, 2023

Overview

Contributes to #108

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id requested a review from rahulghangas February 9, 2023 12:03
@rach-id rach-id requested a review from evan-forbes as a code owner February 9, 2023 12:03
@rach-id rach-id self-assigned this Feb 9, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #109 (04a0fcc) into main (acd5c2f) will decrease coverage by 22.16%.
The diff coverage is n/a.

@@             Coverage Diff             @@
##             main     #109       +/-   ##
===========================================
- Coverage   59.75%   37.60%   -22.16%     
===========================================
  Files           6        9        +3     
  Lines         164      359      +195     
===========================================
+ Hits           98      135       +37     
- Misses         53      209      +156     
- Partials       13       15        +2     
Impacted Files Coverage Δ
evm/evm_client.go 0.00% <ø> (ø)
evm/evm_transaction_opts.go 23.80% <0.00%> (ø)
evm/ethereum_signature.go 90.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rach-id rach-id merged commit f928995 into celestiaorg:main Feb 9, 2023
@rach-id rach-id deleted the remove_evm_client_interface branch February 9, 2023 22:25
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.

3 participants