Closed skosito closed 2 months ago
The recent changes focus on restructuring and enhancing the contract deployment functions within the codebase. Key functions were renamed and updated to improve clarity and functionality, while new parameters were introduced to facilitate better interaction with the protocol's contracts. Additionally, significant updates were made to task parameters for local network interactions, ensuring consistency across contract references and fostering correct operation in the evolving ecosystem.
Files | Change Summary |
---|---|
scripts/worker.ts |
Renamed deploySystemContracts to deployProtocolContracts and added parameters for contract deployment. Updated deployTestContracts to reference protocolContracts . |
tasks/localnet.ts |
Updated parameters for tasks (zevm-call , zevm-withdraw-and-call , evm-call , evm-deposit-and-call ) to reflect new contract addresses, ensuring all tasks use correct references. |
sequenceDiagram
participant User
participant Worker
participant Contract
User->>Worker: Initiate deployProtocolContracts
Worker->>Contract: Deploy contracts with new parameters
Contract-->>Worker: Return contract addresses
Worker-->>User: Output deployed contract addresses
Objective | Addressed | Explanation |
---|---|---|
Error when interacting with localnet (#225) | ❌ | The issue of not printing system contract addresses remains unresolved. |
🐰 In the meadow, contracts bloom,
New names and paths dispel the gloom.
With clearer calls, we leap and play,
In vibrant fields, we find our way!
Hops of code, a joyful tune,
In our world, we'll all commune! 🌼
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?
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 58.10%. Comparing base (
d1746fd
) to head (9a581fd
).
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
yarn localnet
now works for me 👍
Approving following @fadeev feedbacks, as I assume the yarn errors I listed will be solved with the v1/v2 separation
probably you can remove artifacts folder or force recompile to get artifacts from lib folder, but with separation it should be easier yes
@fadeev could you please approve if it works and i will merge this one?
closes: #261
Summary by CodeRabbit
New Features
Bug Fixes
Chores