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

♻️ 4337 compat #27

Merged
merged 2 commits into from
Oct 7, 2023
Merged

♻️ 4337 compat #27

merged 2 commits into from
Oct 7, 2023

Conversation

Vectorized
Copy link
Owner

Description

We can't use the gas opcode in simulation, except where it's immediately followed by one of the call opcodes.

Checklist

Ensure you completed all of the steps below before submitting your pull request:

  • Ran forge fmt?
  • Ran forge snapshot?
  • Ran forge test?

Pull requests with an incomplete checklist will be thrown out.

@Vectorized Vectorized changed the base branch from main to 1.3.0 October 7, 2023 00:33
@Vectorized Vectorized merged commit f9dc02d into 1.3.0 Oct 7, 2023
6 checks passed
@Vectorized Vectorized deleted the 4337-compat branch October 7, 2023 00:35
Vectorized added a commit that referenced this pull request Oct 7, 2023
* ✨ MulticallerWithSigner (#15)

* Tidy

* Update package.json

* ✨ Multicaller with refund and compressed calldata fallback (#18)

* Tidy

* ⚡️ Optimize (#19)

* Optimize

* Edit .gitignore

* ✨ Add ERC1271 support (#21)

* Add more tests

* ⚡️ Remove non-malleability check (#22)

* ⚡️ Optimize (#23)

* New Multicaller deployment

* ♻️ Remove message from MulticallerWithSigner (#24)

* Nit Multicaller comment

* ♻️ Optimizations and fixes (#25)

* 🥢 Nits (#26)

* ♻️ 4337 compat (#27)

* Add review by 0xPhaze

* Update README and credits

* Update README
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