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

introduce 0.7.0 entrypoint #306

Merged
merged 6 commits into from
May 7, 2024
Merged

introduce 0.7.0 entrypoint #306

merged 6 commits into from
May 7, 2024

Conversation

zsluedem
Copy link
Collaborator

No description provided.

.github/scripts/install_solc.sh Show resolved Hide resolved
crates/primitives/src/utils.rs Outdated Show resolved Hide resolved
abigen!(
EntryPointAPI,
r#"[
struct PackedUserOperation {address sender;uint256 nonce;bytes initCode;bytes callData;bytes32 accountGasLimits;uint256 preVerificationGas;bytes32 gasFees;bytes paymasterAndData;bytes signature;}
Copy link
Member

@Vid201 Vid201 Apr 28, 2024

Choose a reason for hiding this comment

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

Nice, we can get packed struct here!

Copy link
Member

@Vid201 Vid201 left a comment

Choose a reason for hiding this comment

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

Good job, looks good!

I have two questinos:

  • bundler spec tests are failing in the CI? Is this related to the problem you mentioned in the Telegram group?
  • we need a way to build docker image for v07? we could expand the docker CI to include branch v07

@zsluedem
Copy link
Collaborator Author

Good job, looks good!

I have two questinos:

  • bundler spec tests are failing in the CI? Is this related to the problem you mentioned in the Telegram group?
  • we need a way to build docker image for v07? we could expand the docker CI to include branch v07
  1. None of bundlers pass all the tests. There are some erros in the tests itself. You can see result in https://bundler-test-results.erc4337.io/v07/history/history.json. All of them have failures.
  2. Yes. We should do that

@Vid201
Copy link
Member

Vid201 commented Apr 29, 2024

Good job, looks good!
I have two questinos:

  • bundler spec tests are failing in the CI? Is this related to the problem you mentioned in the Telegram group?
  • we need a way to build docker image for v07? we could expand the docker CI to include branch v07
  1. None of bundlers pass all the tests. There are some erros in the tests itself. You can see result in https://bundler-test-results.erc4337.io/v07/history/history.json. All of them have failures.
  2. Yes. We should do that

Hmm, ok. This PR looks good now, but it would be good to resolve these 2 things in this PR.

  1. I see, but bundlers are failing only a few tests (less than 10), but in our CI (https://github.com/silius-rs/silius/actions/runs/8875058214/job/24364088083) 100 tests fail.
  2. update docker CI

@zsluedem
Copy link
Collaborator Author

Good job, looks good!
I have two questinos:

  • bundler spec tests are failing in the CI? Is this related to the problem you mentioned in the Telegram group?
  • we need a way to build docker image for v07? we could expand the docker CI to include branch v07
  1. None of bundlers pass all the tests. There are some erros in the tests itself. You can see result in https://bundler-test-results.erc4337.io/v07/history/history.json. All of them have failures.
  2. Yes. We should do that

Hmm, ok. This PR looks good now, but it would be good to resolve these 2 things in this PR.

  1. I see, but bundlers are failing only a few tests (less than 10), but in our CI (https://github.com/silius-rs/silius/actions/runs/8875058214/job/24364088083) 100 tests fail.
  2. update docker CI

OK. Forget to update some args. Locally I got like 6 tests failing

.github/workflows/docker.yml Show resolved Hide resolved
bundler-spec-tests/docker-compose.yml Outdated Show resolved Hide resolved
@zsluedem
Copy link
Collaborator Author

zsluedem commented May 7, 2024

@Vid201 It should be ready now.

Copy link
Member

@Vid201 Vid201 left a comment

Choose a reason for hiding this comment

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

GJ, looks good now!

@Vid201 Vid201 merged commit 3120638 into silius-rs:v07 May 7, 2024
1 of 2 checks passed
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