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.

Sign up for GitHub

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

Jump to bottom

Results to MockEthProvider #5556

Merged
merged 1 commit into from
Nov 24, 2023
Merged

Results to MockEthProvider #5556

merged 1 commit into from
Nov 24, 2023

Conversation

loocapro
Copy link
Contributor

Hi guys,

I d need this mock to return results instead of panicking, would you help me getting this merged?

CC @rkrasiuk @mattsse

@rkrasiuk rkrasiuk added the C-test A change that impacts how or what we test label Nov 24, 2023
@rkrasiuk rkrasiuk added this pull request to the merge queue Nov 24, 2023
Merged via the queue into paradigmxyz:main with commit 3468216 Nov 24, 2023
27 checks passed
@Qumeric
Copy link

Qumeric commented Nov 24, 2023

awesome :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@rkrasiuk rkrasiuk rkrasiuk approved these changes

@rakita rakita Awaiting requested review from rakita rakita is a code owner

@joshieDo joshieDo Awaiting requested review from joshieDo joshieDo is a code owner

Assignees
No one assigned
Labels
C-test A change that impacts how or what we test
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@loocapro @Qumeric @rkrasiuk