Skip to content

Conversation

shohamc1
Copy link
Member

@shohamc1 shohamc1 commented Mar 18, 2025

@shohamc1 shohamc1 marked this pull request as ready for review March 27, 2025 14:31
Copy link
Contributor

@mh0lt mh0lt left a comment

Choose a reason for hiding this comment

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

Hi I have approved the changes - however I think probably the aa exec logic should go under somewhere other than polygon.

However I don't have a suggestion where - so I suggest we leave it as is at the moment.

I'm going to need to do some refactoring when I integrate this into the parallel exec code as I have moved some of the execution functions from the TxTask. Probaby I'll move this code at the same time.

# Conflicts:
#	erigon-lib/common/fixedgas/intrinsic_gas_test.go
@shohamc1 shohamc1 enabled auto-merge (squash) March 29, 2025 03:39
@shohamc1 shohamc1 merged commit f729390 into main Mar 29, 2025
13 checks passed
@shohamc1 shohamc1 deleted the shohamc1/aa-exec branch March 29, 2025 04:07
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.

2 participants