axonweb3 / axon

Axon is a Layer 2 framework of CKB with native cross-chain and interoperability.
https://axonweb3.io
MIT License
65 stars 39 forks source link

fix!: default_max_contract_limit not correctly applied #1653

Closed Flouse closed 7 months ago

Flouse commented 7 months ago

Note: This is a PR to forcerelay-dev branch.

If the hardfork Andromeda is not enabled, use default_max_contract_limit(), because default_max_contract_limit is set to ConsensusConfig by default:

https://github.com/axonweb3/axon/blob/bebd4ca174d95ff21f99b844fba8d9b1221989f6/protocol/src/types/primitive.rs#L440-L460

What is the impact of this PR?

This is a Breaking Change****

CI Settings
### **CI Usage** **Tip**: Check the CI you want to run below, and then comment `/run-ci`. **CI Switch** - [x] Web3 Compatible Tests - [x] OpenZeppelin tests - [x] 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 |
Flouse commented 7 months ago

/run-ci

github-actions[bot] commented 7 months ago

CI tests run on commit:

CI test list: