We read every piece of feedback, and take your input very seriously.
To see all available qualifiers, see our documentation.
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
Shebin and I were looking at the gas discrepancy between our local bundler tests and the Alchemy bundler:
safe-modules/examples/4337-gas-metering/README.md
Line 15 in 32fc6f5
We realised that in the 4337 tests, we consider transaction gas instead of the user operation gas [what we use in benchmarks].
We should adjust the 4337 tests to use the same method. Reference code is available here
The text was updated successfully, but these errors were encountered:
b4a0b60
mmv08
Successfully merging a pull request may close this issue.
Description
Shebin and I were looking at the gas discrepancy between our local bundler tests and the Alchemy bundler:
safe-modules/examples/4337-gas-metering/README.md
Line 15 in 32fc6f5
We realised that in the 4337 tests, we consider transaction gas instead of the user operation gas [what we use in benchmarks].
We should adjust the 4337 tests to use the same method. Reference code is available here
The text was updated successfully, but these errors were encountered: