Closed bitcoinbrisbane closed 2 weeks ago
[!IMPORTANT]
Review skipped
Draft detected.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
The pull request introduces several updates primarily to the README.md
file and smart contracts related to the BroMerge
functionality. Key changes include adjustments to script examples, updates to the contract address for Bro Merge
, and enhancements to the BroMerge
and Eater
contracts, including new functions and event renaming. Additionally, several artifacts related to removed contracts have been deleted. The deployment addresses have also been updated, reflecting these changes.
File | Change Summary |
---|---|
README.md |
- Updated script examples by removing --asset parameter from simple-swap and mint-nft-position commands. - Emphasized the need for FORKING_URL in the blockchain state section. - Updated Bro Merge contract address from 0xe58C... to 0x5031... . |
contracts/core/merge/BroMerge.sol |
- Added _pause() call in the constructor. - Renamed event Eaten to Merged . - Added event RateChanged(uint256 newRate) . - Updated depositAll() to deposit(uint256 amount) and added merge(uint256 amount) and mergeAll() . |
contracts/core/merge/Eater.sol |
- Inherited from Pausable . - Added name() , sweep() , and togglePause() functions. - Updated _merge and receiveApproval functions with whenNotPaused modifier. |
ignition/deployments/chain-8453/artifacts/BroMergeModule#BroMerge.json |
- Updated ABI with renamed event Eaten to Merged and added RateChanged . - Updated deposit function signature. |
ignition/deployments/chain-8453/artifacts/BroMergeModule#BroMerge.dbg.json |
- Updated buildInfo reference. |
ignition/deployments/chain-8453/artifacts/ChaosModule#CHAOS.json |
- Removed contract artifact for CHAOS . |
ignition/deployments/chain-8453/artifacts/FarmModule#Farm.json |
- Removed contract artifact for Farm . |
ignition/deployments/chain-8453/artifacts/MigrationModule#Migration.json |
- Removed contract artifact for Migration . |
ignition/deployments/chain-8453/deployed_addresses.json |
- Updated BroMergeModule#BroMerge address and removed entries for Farm , Migration , and Chaos . |
ignition/deployments/chain-8453/journal.jsonl |
- Updated entries for MigrationModule , ChaosModule , and BroMergeModule with new transaction details. |
ignition/modules/BroMerge.ts |
- Added constant rate and a call to setRate in BroMergeModule . |
test/GenericMerge.test.ts |
- Added tests for ownership and pause functionality, including tests for sweep and toggling pause state. |
README.md
regarding command-line instructions and contract addresses.README.md
that align with the focus on updating the contract address for Bro Merge
.SC
π° In the meadow, changes bloom,
Scripts and contracts find new room.
With pauses set and rates defined,
TheBroMerge
hops, well-aligned.
So letβs cheer for code so bright,
A joyful leap into the night! π
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Summary by CodeRabbit
Release Notes
New Features
Eater
contract for enhanced token management:sweep()
andtogglePause()
.Eater
contract, preventing actions when paused.Bug Fixes
Bro Merge
to ensure correct interactions.Documentation
README.md
to reflect changes in script commands and contract addresses.Tests
GenericMerge
contract.