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

chore(CI): add eth_getStorageAt, eth_getProof cases #1632

Merged
merged 2 commits into from
Dec 12, 2023

Conversation

sunchengzhu
Copy link
Collaborator

@sunchengzhu sunchengzhu commented Dec 8, 2023

Update hardfork_test.yml

update hardfork_test ref

What this PR does / why we need it?

This PR for add cases: axonweb3/axon-hardfork-test@fe6613a

What is the impact of this PR?

No Breaking Change

PR relation:

Compare test code

https://github.com/axonweb3/axon-hardfork-test/compare/b1f768de8be10604cad882474c0dcfa19c0a93ea..69293ac366991cf5830ab8366a85d72449daeadc

test/verifyProof.ts

https://github.com/axonweb3/axon-hardfork-test/blob/69293ac366991cf5830ab8366a85d72449daeadc/test/verifyProof.ts

CI Settings

CI Usage

Tip: Check the CI you want to run below, and then comment /run-ci.

CI Switch

  • Web3 Compatible Tests
  • OpenZeppelin tests
  • v3 Core Tests

CI Description

CI Name Description
Web3 Compatible Test Test the Web3 compatibility of Axon
v3 Core Test Run the compatibility tests provided by Uniswap V3
OpenZeppelin tests Run the compatibility tests provided by OpenZeppelin, including OCT 1-5 | 6-10 | 11 | 12-15 | 16-19

Update hardfork_test.yml

update hardfork_test ref
@Flouse Flouse requested review from driftluo and blckngm and removed request for yangby-cryptape December 9, 2023 03:12
@Flouse Flouse added this pull request to the merge queue Dec 12, 2023
Merged via the queue into main with commit 7d8de68 Dec 12, 2023
23 checks passed
@KaoImin KaoImin deleted the scz/add-eth_getProof-cases branch December 13, 2023 07:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants