Skip to content

ci: add Rust quality checks - #56

Merged
bmuddha merged 1 commit into
devfrom
ci-rust-checks
Aug 12, 2026
Merged

ci: add Rust quality checks#56
bmuddha merged 1 commit into
devfrom
ci-rust-checks

Conversation

@lucacillario

Copy link
Copy Markdown
Contributor

What changed

Add independent pull-request workflows for formatting, Clippy, and nextest. Configure Rust and Solana toolchains plus cached Cargo and Solana build artifacts.

Closes #39

Impact

Pull requests containing these workflows report all three standard Rust checks independently. The change affects CI only.

Validation

  • Parsed all three workflow files as YAML.
  • cargo fmt --check passed on the complete replicator stack.
  • git diff --check passed.
  • Clippy reached the existing Keeper SBF build locally; the installed Solana 3.0.11 platform toolchain is too old for edition 2024. CI installs Solana 4.0.3.
  • Nextest is installed and exercised by the CI workflow.

Reviewer notes

This branch is stacked on replicator, the first point where the complete engine workspace exists. Existing stack branches remain unchanged.

Swatinem/rust-cache@v2 was chosen instead of sccache because it integrates directly with GitHub Actions’ persistent cache and efficiently restores Cargo dependencies and build artifacts across ephemeral runners while keeping the CI setup simpler.

## What changed

Add independent pull-request workflows for formatting, Clippy, and nextest. Configure Rust and Solana toolchains plus cached Cargo and Solana build artifacts.

Closes #39

## Impact

Pull requests containing these workflows report all three standard Rust checks independently. The change affects CI only.

## Validation

- Parsed all three workflow files as YAML.
- `cargo fmt --check` passed on the complete `replicator` stack.
- `git diff --check` passed.
- Clippy reached the existing Keeper SBF build locally; the installed Solana 3.0.11 platform toolchain is too old for edition 2024. CI installs Solana 4.0.3.
- Nextest is installed and exercised by the CI workflow.

## Reviewer notes

This branch is stacked on `replicator`, the first point where the complete engine workspace exists. Existing stack branches remain unchanged.
@coderabbitai

coderabbitai Bot commented Aug 12, 2026

Copy link
Copy Markdown

Warning

Review limit reached

@lucacillario, you've reached your PR review limit, so we couldn't start this review.

Next review available in: 7 minutes

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

How can I continue?

After more reviews become available, a review can be triggered using the @coderabbitai review command as a PR comment. Alternatively, push new commits to this PR.

To avoid repeated limits, reduce automatic review volume by pausing incremental auto-reviews earlier, using label-based review opt-in, excluding WIP or generated PR titles, or requesting reviews manually when the PR is ready. If your team needs uninterrupted high-volume reviews, an organization admin can enable usage-based reviews.

How do review limits work?

CodeRabbit enforces per-developer PR review limits for each organization. Most developers receive the normal plan review availability.

For paid Pro and Pro+ PR reviews, CodeRabbit uses adaptive limits for sustained high-volume activity. When a developer's recent PR review activity reaches the 95th percentile or higher among CodeRabbit users, additional reviews become available more gradually as earlier reviews age out of the rolling window.

Please refer docs for additional details.

Review details
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro Plus

Run ID: 6143a25e-8625-40ae-a3ef-66f6b21d0773

📥 Commits

Reviewing files that changed from the base of the PR and between 0e9d3d2 and 0a98a65.

📒 Files selected for processing (3)
  • .github/workflows/ci-fmt.yml
  • .github/workflows/ci-lint.yml
  • .github/workflows/ci-test.yml

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.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@bmuddha
bmuddha merged commit e48b431 into dev Aug 12, 2026
3 of 4 checks passed
@bmuddha
bmuddha deleted the ci-rust-checks branch August 12, 2026 15:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Configure full Rust CI workflow

2 participants