Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

runtime-sdk/modules/evm: Revert "drop signed queries in new contracts" #1500

Merged
merged 1 commit into from
Sep 20, 2023

Conversation

kostko
Copy link
Member

@kostko kostko commented Sep 18, 2023

Given further thought this seems too much like a foot gun and may need some more thought.

Given further thought this seems too much like a foot gun.
@codecov
Copy link

codecov bot commented Sep 18, 2023

Codecov Report

Merging #1500 (ae0dd9c) into main (b9bea98) will decrease coverage by 0.04%.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main    #1500      +/-   ##
==========================================
- Coverage   60.42%   60.39%   -0.04%     
==========================================
  Files         138      138              
  Lines        9757     9734      -23     
==========================================
- Hits         5896     5879      -17     
+ Misses       3819     3813       -6     
  Partials       42       42              
Files Changed Coverage Δ
runtime-sdk/modules/evm/src/backend.rs 70.00% <ø> (+0.04%) ⬆️
runtime-sdk/modules/evm/src/state.rs 91.66% <ø> (-1.44%) ⬇️
runtime-sdk/modules/evm/src/types.rs 66.66% <ø> (ø)
runtime-sdk/modules/evm/src/lib.rs 64.79% <100.00%> (+0.58%) ⬆️

@pro-wh
Copy link
Contributor

pro-wh commented Sep 18, 2023

#1481 cross reference

@kostko kostko merged commit 54f5df8 into main Sep 20, 2023
26 of 27 checks passed
@kostko kostko deleted the kostko/feature/evm-revert-drop-signed-queries branch September 20, 2023 06:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants