Closed skosito closed 3 months ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Base branches to auto review (1)
* mainPlease 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 changes introduce a more robust development workflow focused on Solidity smart contracts. They include configuration and workflow adjustments for efficient testing, linting, coverage reporting, automatic reviews, and publishing. Significant modifications include enhancements to GitHub Actions workflows, a smart contract for managing omnichain interactions, interface updates, and additions to supporting tools and configurations.
Files/Groups | Summary |
---|---|
.coderabbit.yaml |
Added configuration instructions for code reviews |
.github/workflows/coverage.yaml |
Introduced a GitHub Actions workflow for test coverage with Codecov uploads |
.github/workflows/generated-files.yaml .github/workflows/lint.yaml .github/workflows/publish-npm.yaml .github/workflows/semantic-pr.yaml .github/workflows/test.yaml |
Adjusted workflow triggers and conditions |
.gitignore |
Added entries for code coverage files |
.solcover.js |
Added configuration for Solidity contract testing |
contracts/prototypes/zevm/GatewayZEVM.sol test/prototypes/GatewayZEVM.spec.ts typechain-types/contracts/prototypes/index.ts typechain-types/contracts/prototypes/zevm/GatewayZEVM.ts typechain-types/contracts/prototypes/zevm/index.ts typechain-types/factories/contracts/prototypes/zevm/GatewayZEVM__factory.ts typechain-types/factories/contracts/prototypes/zevm/index.ts |
Introduced the GatewayZEVM contract, its tests, and related typechain & factory files |
contracts/zevm/interfaces/IZRC20.sol typechain-types/contracts/zevm/interfaces/IZRC20.ts |
Renamed PROTOCOL_FEE to PROTOCOL_FLAT_FEE and updated function signatures |
docs/src/contracts/zevm/interfaces/IZRC20.sol/interface.IZRC20.md |
Updated documentation for renamed PROTOCOL_FEE function |
hardhat.config.ts |
Added imports for solidity-coverage and hardhat-gas-reporter |
package.json |
Updated dependencies, removed Node engine requirement, adjusted lint script |
pkg/contracts/zevm/interfaces/izrc20.sol/izrc20.go pkg/contracts/zevm/systemcontract.sol/systemcontract.go pkg/contracts/zevm/testing/systemcontractmock.sol/systemcontractmock.go |
Updated ABI definitions and function signatures |
scripts/generate_addresses_types.ts |
Improved type safety for mainnet and testnet array typing |
🎉 Amid the code and logic's dance, A rabbit pranced, full of chance, With configs set and workflows tight, It brought the changes, day and night. Solidity sang, reviews in flight, In code, we found the future bright. 🐇✨
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?
Attention: Patch coverage is 93.33333%
with 2 lines
in your changes missing coverage. Please review.
Project coverage is 57.46%. Comparing base (
b9e5bfd
) to head (fe21cdc
).
Files | Patch % | Lines |
---|---|---|
contracts/prototypes/zevm/GatewayZEVM.sol | 90.47% | 2 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
@coderabbitai review
Seems like the restricted path for @coderabbitai in the config doesn't work. It should not look at Go code
[!TIP] For best results, initiate chat on the files or code changes.
@lumtis, it appears that the current configuration does not explicitly exclude Go code from reviews. If you wish to prevent CodeRabbit from reviewing Go code, you can add a path filter in the .coderabbit.yaml
configuration file to exclude Go files. For example, you could add !**/*.go
to the path_filters
section to exclude all Go files from being reviewed.
Here is how you could update the configuration:
reviews:
path_filters:
- "!**/*.go" # Exclude all Go files
Adding this line will ensure that Go files are not included in the reviews. If you need further assistance with this configuration, please let me know!
Initializes GatewayZEVM and implements specs for inbound:
Summary by CodeRabbit
New Features
GatewayZEVM
Solidity smart contract for handling omnichain smart contract interactions and ZRC20 token withdrawals.Enhancements
Bug Fixes
PROTOCOL_FEE
toPROTOCOL_FLAT_FEE
in theIZRC20
interface.Documentation
IZRC20
interface.Chores
.gitignore
to exclude coverage reports and specific script files.