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

feat(x/evm): MsgServer #290

Merged
merged 8 commits into from
Feb 24, 2023
Merged

feat(x/evm): MsgServer #290

merged 8 commits into from
Feb 24, 2023

Conversation

itsdevbear
Copy link
Member

No description provided.

@codecov
Copy link

codecov bot commented Feb 24, 2023

Codecov Report

Merging #290 (434d801) into main (bcb0ea0) will increase coverage by 4.53%.
The diff coverage is 2.56%.

❗ Current head 434d801 differs from pull request most recent head ee48816. Consider uploading reports for the commit ee48816 to get more accurate results

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #290      +/-   ##
==========================================
+ Coverage   72.33%   76.86%   +4.53%     
==========================================
  Files          67       69       +2     
  Lines        2653     2689      +36     
==========================================
+ Hits         1919     2067     +148     
+ Misses        682      564     -118     
- Partials       52       58       +6     
Flag Coverage Δ
integration 6.06% <0.00%> (?)
unit 71.29% <2.56%> (-1.05%) ⬇️
Impacted Files Coverage Δ
x/evm/keeper/msg_server.go 0.00% <0.00%> (ø)
x/evm/types/tx.go 33.33% <0.00%> (-6.21%) ⬇️
x/evm/types/update_params.go 0.00% <0.00%> (ø)
x/evm/keeper/processor.go 82.75% <100.00%> (-1.12%) ⬇️
testutil/app/cmd/stargazerd/cmd/root.go 74.62% <0.00%> (+74.62%) ⬆️

return nil, err
}
// TODO: should receipt just be bytes? not string?
etr.Receipt = string(bz)
Copy link
Contributor

Choose a reason for hiding this comment

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

bytes

@itsdevbear
Copy link
Member Author

switched to bytes

@itsdevbear itsdevbear merged commit 5712707 into main Feb 24, 2023
@itsdevbear itsdevbear deleted the msg-server branch May 3, 2023 14:33
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