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

fix: Adds return types where missing in eth.ts #2265

Merged
merged 4 commits into from
May 16, 2024

Conversation

konstantinabl
Copy link
Collaborator

Description:

This PR adds return types to any functions missing them in the eth.ts file
Related issue(s):

Fixes #2264

@konstantinabl konstantinabl linked an issue Mar 29, 2024 that may be closed by this pull request
@konstantinabl konstantinabl marked this pull request as draft March 29, 2024 13:54
@konstantinabl konstantinabl force-pushed the 2264-add-return-types-in-ethts-where-missing branch from 2f69ca0 to b62aa2b Compare April 24, 2024 08:53
Copy link

github-actions bot commented Apr 24, 2024

Tests

    2 files  147 suites   13s ⏱️
819 tests 818 ✔️ 1 💤 0
831 runs  830 ✔️ 1 💤 0

Results for commit 89995b9.

♻️ This comment has been updated with latest results.

Copy link

github-actions bot commented Apr 24, 2024

Acceptance Tests

     26 files     406 suites   39m 12s ⏱️
   582 tests    573 ✔️   3 💤   6
1 653 runs  1 619 ✔️ 19 💤 15

Results for commit 89995b9.

♻️ This comment has been updated with latest results.

@konstantinabl konstantinabl marked this pull request as ready for review April 24, 2024 12:45
@konstantinabl konstantinabl force-pushed the 2264-add-return-types-in-ethts-where-missing branch 2 times, most recently from c3c79a9 to 8cbb200 Compare April 26, 2024 07:23
Copy link
Collaborator

@georgi-l95 georgi-l95 left a comment

Choose a reason for hiding this comment

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

LG

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
@AlfredoG87 or @ebadiere to check also

Copy link
Collaborator

@AlfredoG87 AlfredoG87 left a 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>
@konstantinabl konstantinabl force-pushed the 2264-add-return-types-in-ethts-where-missing branch from 8cbb200 to 89995b9 Compare May 14, 2024 09:42
Copy link

Quality Gate Passed Quality Gate passed

Issues
5 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@konstantinabl konstantinabl requested a review from Nana-EC May 16, 2024 11:54
@konstantinabl konstantinabl merged commit 2f80289 into main May 16, 2024
33 checks passed
@konstantinabl konstantinabl deleted the 2264-add-return-types-in-ethts-where-missing branch May 16, 2024 12:13
quiet-node pushed a commit that referenced this pull request May 20, 2024
* 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>
@quiet-node quiet-node mentioned this pull request May 20, 2024
2 tasks
quiet-node pushed a commit that referenced this pull request May 20, 2024
* 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>
quiet-node added a commit that referenced this pull request May 20, 2024
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>
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.

Add return types in eth.ts where missing
4 participants