Skip to content

feat: add ckb_blake2b precompile contract#1555

Merged
Flouse merged 3 commits intomainfrom
add-ckb-blake2b-precompile
Nov 14, 2023
Merged

feat: add ckb_blake2b precompile contract#1555
Flouse merged 3 commits intomainfrom
add-ckb-blake2b-precompile

Conversation

@KaoImin
Copy link
Copy Markdown
Contributor

@KaoImin KaoImin commented Nov 14, 2023

What this PR does / why we need it?

This PR add ckb_blake2b hash precompile contract.
The gas calculation is same as sha2-256.

What is the impact of this PR?

No Breaking Change

PR relation:

  • Ref #
CI Settings

CI Usage

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

CI Switch

  • Web3 Compatible Tests
  • OCT 1-5 And 12-15
  • OCT 6-10
  • OCT 11
  • OCT 16-19
  • 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
OCT 1-5 | 6-10 | 11 | 12-15 | 16-19 Run the compatibility tests provided by OpenZeppelin

@KaoImin KaoImin requested a review from a team as a code owner November 14, 2023 03:49
@KaoImin KaoImin requested review from blckngm, chaoticlonghair, driftluo and wenyuanhust and removed request for wenyuanhust November 14, 2023 03:49
driftluo
driftluo previously approved these changes Nov 14, 2023
@Flouse Flouse requested a review from ashuralyk November 14, 2023 06:51
@KaoImin KaoImin enabled auto-merge November 14, 2023 07:26
blckngm
blckngm previously approved these changes Nov 14, 2023
Flouse

This comment was marked as resolved.

@Flouse Flouse dismissed stale reviews from blckngm and driftluo via 6e8529d November 14, 2023 10:31
@Flouse Flouse disabled auto-merge November 14, 2023 10:31
@Flouse Flouse merged commit ea9ae7e into main Nov 14, 2023
@KaoImin KaoImin deleted the add-ckb-blake2b-precompile branch November 14, 2023 16:37
@Flouse Flouse added t:feature and removed feature labels Nov 17, 2023
KaoImin pushed a commit that referenced this pull request Nov 17, 2023
* feat: add `ckb_blake2b` precompile contract
* add unit test case
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