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.

Sign up for GitHub

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

Jump to bottom

feat: enable get header precompile #1649

Merged
merged 1 commit into from
Dec 21, 2023
Merged

feat: enable get header precompile #1649

merged 1 commit into from
Dec 21, 2023

Conversation

blckngm
Copy link
Contributor

@blckngm blckngm commented Dec 21, 2023 •
edited by Flouse
Loading

What this PR does / why we need it?

This PR enables the GetHeader precompile.

What is the impact of this PR?

No Breaking Change

CI 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

@blckngm blckngm requested a review from a team as a code owner December 21, 2023 05:56
@Flouse Flouse requested review from wenyuanhust and removed request for Simon-Tl December 21, 2023 06:00
@Flouse
Copy link
Contributor

Flouse commented Dec 21, 2023

/run-ci

@github-actions GitHub Actions
Copy link

CI tests run on commit:

CI test list:

  • openzeppelin_test_1_5_and_12_15.yml
  • openzeppelin_test_11.yml
  • openzeppelin_test_6_10.yml
  • openzeppelin_test_16_19.yml
  • v3_core_test.yml
  • web3_compatible.yml

Please check ci test results later.

@Flouse
Copy link
Contributor

Flouse commented Dec 21, 2023

sync test passed.
https://github.com/axonweb3/axon/actions/runs/7284552825/job/19850123063#step:7:68

@Flouse Flouse added this pull request to the merge queue Dec 21, 2023
Merged via the queue into main with commit 077023a Dec 21, 2023
34 of 44 checks passed
@KaoImin KaoImin deleted the enable-get-header-precompile branch December 28, 2023 05:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@KaoImin KaoImin KaoImin approved these changes

@wenyuanhust wenyuanhust wenyuanhust approved these changes

@ashuralyk ashuralyk ashuralyk approved these changes

@Flouse Flouse Flouse approved these changes

Assignees
No one assigned
Labels
feature
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@blckngm @Flouse @ashuralyk @wenyuanhust @KaoImin