Closed Pitasi closed 2 weeks ago
[!WARNING]
Rate limit exceeded
@Pitasi has exceeded the limit for the number of commits or files that can be reviewed per hour. Please wait 53 minutes and 57 seconds before requesting another review.
How to resolve this issue?
After the wait time has elapsed, a review can be triggered using the `@coderabbitai review` command as a PR comment. Alternatively, push new commits to this PR. We recommend that you space out your commits to avoid hitting the rate limit.How do rate limits work?
CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our [FAQ](https://coderabbit.ai/docs/faq) for further information.Commits
Files that changed from the base of the PR and between 5b06b93aabc52ca075a15877e7ef2c88c20cc6a9 and 5c7512ca1da1ec0d215b9fb8df54685710bd6695.
The changes primarily focus on enhancing the deployment and management of CosmWasm contracts and integrating Ethereum analyzer functionalities. New dependencies are added to the Cargo.toml
file for contract analysis. A justfile
is introduced for managing contract compilation and deployment. The localnet
block in the justfile
is updated to deploy the Ethereum analyzer contract, and the env.ts
file in the Spaceward project sets a default contract address for the Ethereum analyzer.
File Path | Change Summary |
---|---|
contracts/Cargo.toml |
Added dependencies: basic-analyzer and ethereum-analyzer with specific paths and versions, marked with the feature "library". |
contracts/justfile |
Introduced commands for managing and compiling CosmWasm contracts, including tasks for regenerating schemas and compiling contracts. |
justfile |
Added post_start function within the localnet block, modified command sequence, and added deploy-contract script. |
spaceward/src/env.ts |
Updated default value for ethereumAnalyzerContract to include a specific contract address if not provided. |
sequenceDiagram
participant Developer
participant Justfile
participant Localnet
participant EthereumAnalyzerContract
participant SpacewardEnv
Developer->>Justfile: Run `just localnet`
Justfile->>Localnet: Start local blockchain
Localnet->>EthereumAnalyzerContract: Check if contract is deployed
alt Contract not deployed
Localnet->>Justfile: Deploy contract
Justfile->>EthereumAnalyzerContract: Compile and deploy contract
end
EthereumAnalyzerContract->>Localnet: Contract deployed
Localnet->>Developer: Localnet ready
Developer->>SpacewardEnv: Access Ethereum analyzer contract
SpacewardEnv->>Developer: Provide contract address
Objective | Addressed | Explanation |
---|---|---|
Deploy Ethereum analyzer contract during just localnet (#376) |
✅ | |
Compile the Rust code into the wasm binary (#376) | ✅ | |
Run a tx to store binary on chain (#376) | ✅ | |
Instantiate the contract on chain (#376) | ✅ | |
Ensure the contract address is deterministic and hardcoded in env.ts (#376) |
✅ |
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?
Hey @Pitasi and thank you for opening this pull request! 👋🏼
It looks like you forgot to add a changelog entry for your changes. Make sure to add a changelog entry in the 'CHANGELOG.md' file.
Tried locally just localnet
My version of sed is not working with sed -i 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml
, but working with
sed -i '' -e 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml
I have Mac Sonoma 14.4.1, text_cmds-165.100.8 (it's sed version in MacOs, from here https://opensource.apple.com/releases/)
After fixing have error unknown command "add-genesis-space" for "genesis"
Tried locally just localnet
- My version of sed is not working with
sed -i 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml
, but working withsed -i '' -e 's/cors_allowed_origins = \[\]/cors_allowed_origins = ["*"]/' ~/.warden/config/config.toml
I have Mac Sonoma 14.4.1, text_cmds-165.100.8 (it's sed version in MacOs, from here https://opensource.apple.com/releases/)
I fixed that issue on main, I now rebased this PR so it contains the fix :)
- After fixing have error
unknown command "add-genesis-space" for "genesis"
I think you have a really old version of wardend. You can update it with just install
.
Adds a justfile for building our contracts, it also automatically deploys "ethereum-analyzer" as part of our localnet setup (requires Docker and
jq
)closes #376
Summary by CodeRabbit
New Features
Improvements