Skip to content
New issue

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

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

Already on GitHub? # to your account

Complete eth_getStorageAt implementation #361

Merged
merged 3 commits into from
Jul 21, 2022
Merged

Conversation

lukelee-sl
Copy link
Member

Description:
Implement the eth_getStorageAt call as an unsupported method.

Related issue(s):

Fixes #355

Notes for reviewer:

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@lukelee-sl lukelee-sl requested review from shemnon and Nana-EC July 20, 2022 17:12
@codecov-commenter
Copy link

codecov-commenter commented Jul 20, 2022

Codecov Report

Merging #361 (42d5f52) into main (44d85ef) will decrease coverage by 0.14%.
The diff coverage is 0.00%.

@@            Coverage Diff             @@
##             main     #361      +/-   ##
==========================================
- Coverage   62.35%   62.21%   -0.15%     
==========================================
  Files           9        9              
  Lines         866      868       +2     
  Branches      142      142              
==========================================
  Hits          540      540              
- Misses        288      290       +2     
  Partials       38       38              
Impacted Files Coverage Δ
packages/relay/src/lib/eth.ts 65.22% <0.00%> (-0.34%) ⬇️

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 44d85ef...42d5f52. Read the comment docs.

Signed-off-by: Luke Lee <luke.lee@swirldslabs.com>
@lukelee-sl lukelee-sl force-pushed the 355-complete-getStorageAt branch from 55db696 to 20c1f9d Compare July 20, 2022 17:37
Signed-off-by: Luke Lee <luke.lee@swirldslabs.com>
@Nana-EC Nana-EC added enhancement New feature or request good first issue Good for newcomers P2 labels Jul 20, 2022
@Nana-EC Nana-EC added this to the 0.5.0 milestone Jul 20, 2022
Nana-EC
Nana-EC previously approved these changes Jul 20, 2022
Copy link
Collaborator

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

Nice first PR.
2 non-blocking nits

Nana-EC
Nana-EC previously approved these changes Jul 21, 2022
Copy link
Collaborator

@Nana-EC Nana-EC left a comment

Choose a reason for hiding this comment

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

LGTM

Signed-off-by: lukelee-sl <luke.lee@swirldslabs.com>
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@lukelee-sl lukelee-sl requested a review from Nana-EC July 21, 2022 04:46
@lukelee-sl
Copy link
Member Author

lukelee-sl commented Jul 21, 2022

@Nana-EC one more approval please. For whatever reason, the email used when signing my last 2 commits was incorrect. I've addressed the issue and resubmitted.

@lukelee-sl lukelee-sl merged commit cd003b2 into main Jul 21, 2022
@lukelee-sl lukelee-sl deleted the 355-complete-getStorageAt branch July 21, 2022 14:52
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
enhancement New feature or request good first issue Good for newcomers P2
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Complete eth_getStorageAt implementation
3 participants