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

tests: itests: log evm errors #10157

Closed
wants to merge 4 commits into from
Closed

Conversation

snissn
Copy link
Contributor

@snissn snissn commented Feb 1, 2023

Related Issues

  • none

Proposed Changes

quality of life improvement for evm tests - print evm error and bubble up evm error in a helper

Additional Info

Checklist

Before you mark the PR ready for review, please make sure that:

  • Commits have a clear commit message.
  • PR title is in the form of of <PR type>: <area>: <change being made>
    • example: fix: mempool: Introduce a cache for valid signatures
    • PR type: fix, feat, build, chore, ci, docs, perf, refactor, revert, style, test
    • area, e.g. api, chain, state, market, mempool, multisig, networking, paych, proving, sealing, wallet, deps
  • New features have usage guidelines and / or documentation updates in
  • Tests exist for new functionality or change in behavior
  • CI is green

@snissn snissn changed the base branch from mikers/create2test to snissn/recursivetest February 1, 2023 03:23
@snissn snissn closed this Feb 1, 2023
@snissn snissn deleted the mikers/logEVMErrors branch February 1, 2023 03:26
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.

1 participant