-
Notifications
You must be signed in to change notification settings - Fork 77
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
fix: Adds return types where missing in eth.ts #2265
fix: Adds return types where missing in eth.ts #2265
Conversation
2f69ca0
to
b62aa2b
Compare
c3c79a9
to
8cbb200
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LG
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@AlfredoG87 or @ebadiere to check also
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, nice to have typed return types
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
…cies Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
8cbb200
to
89995b9
Compare
Quality Gate passedIssues Measures |
* Merges main Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> * Removes unused import Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> * Adds return types to eth.ts; Refactors fee history to remove redundancies Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> * Adds transaction receipt interface Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> --------- Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com>
* Merges main Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> * Removes unused import Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> * Adds return types to eth.ts; Refactors fee history to remove redundancies Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> * Adds transaction receipt interface Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> --------- Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com>
fix: Adds return types where missing in eth.ts (#2265) * Merges main * Removes unused import * Adds return types to eth.ts; Refactors fee history to remove redundancies * Adds transaction receipt interface --------- Signed-off-by: Konstantina Blazhukova <konstantina.blajukova@gmail.com> Signed-off-by: Logan Nguyen <logan.nguyen@swirldslabs.com> Co-authored-by: konstantinabl <konstantina.blajukova@gmail.com>
Description:
This PR adds return types to any functions missing them in the eth.ts file
Related issue(s):
Fixes #2264