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

Problem: verbose logs display with FATAL option (fix #320) #333

Closed
wants to merge 1 commit into from

Conversation

leejw51crypto
Copy link
Contributor

@leejw51crypto leejw51crypto commented Jul 21, 2021

Solution: replace suplog with tendermint one
only display evm log in debug log-level

add my script

increase amount for metamask

add run

amount ok

hide log

show info

my logger

hook log

revive eth log

tidy up

use suplog

log replace ok

removed suplog

tidy up

tidy up

fix compile

remove sh

tidy up

tidy up
@codecov
Copy link

codecov bot commented Jul 21, 2021

Codecov Report

Merging #333 (e61cbcb) into main (0276f34) will decrease coverage by 6.62%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #333      +/-   ##
==========================================
- Coverage   47.38%   40.76%   -6.63%     
==========================================
  Files          46       38       -8     
  Lines        4611     3454    -1157     
==========================================
- Hits         2185     1408     -777     
+ Misses       2348     1993     -355     
+ Partials       78       53      -25     
Impacted Files Coverage Δ
x/evm/handler.go 4.00% <0.00%> (-0.17%) ⬇️
x/evm/types/utils.go 60.00% <0.00%> (ø)
app/export.go
ethereum/rpc/pubsub/pubsub.go
app/ante/eth.go
app/app.go
app/test_helpers.go
crypto/hd/algorithm.go
crypto/ethsecp256k1/ethsecp256k1.go

@fedekunze
Copy link
Contributor

see #343 for an alternative

@leejw51crypto
Copy link
Contributor Author

modify based upon 343

mmsqe pushed a commit to mmsqe/ethermint that referenced this pull request Sep 7, 2023
* Problem: don't support native action in statedb

* Update CHANGELOG.md

Signed-off-by: yihuang <huang@crypto.com>

* fix deps

* fix commit

* fix build

* fix build

* fix lint

---------

Signed-off-by: yihuang <huang@crypto.com>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants