FuelLabs / fuel-vm

Fuel v2 interpreter in Rust
Other
358 stars 88 forks source link

Add `block_transaction_size_limit` to `ConsensusParameters` #821

Closed rafal-ch closed 2 months ago

rafal-ch commented 2 months ago

Partial implementation of https://github.com/FuelLabs/fuel-core/issues/2133.

This PR adds a new version (V2) of the ConsensusParameters which includes the block_transaction_size_limit parameter.

Linked Issues/PRs

https://github.com/FuelLabs/fuel-core/pull/2188

Checklist

Before requesting review

After merging, notify other teams

[Add or remove entries as needed]

rafal-ch commented 2 months ago

LGTM, a test which asserts SettingBlockTransactionSizeLimitNotSupported is thrown would be nice :)

Test added in https://github.com/FuelLabs/fuel-vm/pull/821/commits/d07fbc3dbf1a1965fc35450d6d801874c1ffe532

acerone85 commented 2 months ago

Checked the code again, LGTM.