build: remove unused dockerfile and docker-compose#1649
Conversation
📝 WalkthroughWalkthroughAll Docker build and runtime instructions for Go-based components were removed from the Dockerfile, leaving only the Node.js build environment setup. The docker-compose.yml file, previously defining multiple interconnected services, was deleted. The localnet.just script was updated to add an optional contract deployment flag and two new governance parameters. Changes
Sequence Diagram(s)sequenceDiagram
participant User
participant localnet.just
User->>localnet.just: start(deploy-contracts)
alt deploy-contracts == "false"
localnet.just-->>User: Skip contract deployment
else deploy-contracts == "true"
localnet.just->>localnet.just: Deploy contracts
end
Estimated code review effort🎯 3 (Moderate) | ⏱️ ~15 minutes Possibly related PRs
Suggested labels
Suggested reviewers
Note ⚡️ Unit Test Generation is now available in beta!Learn more here, or try it out under "Finishing Touches" below. 📜 Recent review detailsConfiguration used: .coderabbit.yaml 📒 Files selected for processing (3)
💤 Files with no reviewable changes (2)
🧰 Additional context used🧠 Learnings (2)📓 Common learnings📚 Learning: in `solidity/orders/script/deploy.s.sol`, environment variables for salts (`registry_salt` and `fact...Applied to files:
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
🔇 Additional comments (3)
✨ Finishing Touches🧪 Generate unit tests
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
SupportNeed help? Create a ticket on our support page for assistance with any issues or questions. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
Documentation and Community
|
removing some old stuff, and avoiding deploying cosmwasm contracts by default on localnet since we are not using them 99.9% of the times