Skip to content

style(agents): format the partial profile test - #81

Merged
tee8z merged 1 commit into
mainfrom
fix/fmt-profile-tests
Oct 26, 2026
Merged

tee8z merged 1 commit into
mainfrom
fix/fmt-profile-tests

Conversation

@tee8z

@tee8z tee8z commented Oct 23, 2026 •

Copy link
Copy Markdown
Collaborator

What

#80 merged with one unformatted test in crates/buzz-sandbox-reaper/src/profile_tests.rs, so just fmt-check fails the Rust Lint and GPUI Format and lint jobs on main. This is cargo fmt --all with no other change. cargo fmt --all --check passes.

Summary by CodeRabbit

  • Tests
    • Reformatted the partial-profile validation test cases for readability. Tested inputs and expected errors are unchanged.

@coderabbitai

coderabbitai Bot commented Oct 23, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

No actionable comments were generated in the recent review.

Recent review info
Run configuration
Commits

Reviewing files that changed from the base of the PR and between 5cc2f1a and 4b31928.

Files selected for processing: 1
  • crates/buzz-sandbox-reaper/src/profile_tests.rs

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

Walkthrough

The partial-profile validation test now formats four invalid-input cases as multiline tuples. The inputs and expected validation errors remain unchanged.

Changes: Partial-profile validation tests

Layer / File(s) Summary
Reformat invalid-input tuples
crates/buzz-sandbox-reaper/src/profile_tests.rs
The four invalid-input test tuples use multiline formatting. Their inputs and expected errors are unchanged.

Priority: Low

Estimated code review effort: 1 (Trivial) | ~2 minutes

Change: Other

Possibly related PRs

Merge Risk: Minimal · up to 4b319

This formatting-only change is ready to merge.

Pre-merge checks: Passed 5
Check name Status Explanation
Description Check Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check Passed The title clearly and concisely describes the formatting-only change to the partial profile test. It matches the main change in the pull request.
Docstring Coverage Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%. Docstring coverage is scoped to functions touched by this diff. Analyzed 1 functions across 1 files.
Linked Issues check Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check Passed Check skipped because no linked issues were found for this pull request.
Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@tee8z

tee8z commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator Author

Reviewed the complete diff against main: it only applies rustfmt to four test tuples and preserves their values and assertions. cargo fmt --all --check passes. CI is green; no code blockers found.

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown
✅ Action performed

Review finished.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@tee8z

tee8z commented Oct 26, 2026

Copy link
Copy Markdown
Collaborator Author

@coderabbitai The review of current head 4b31928 is complete with no findings, and CI is green. Please run the normal approval check for this reviewed head, preserving all review and pre-merge requirements. If approvals are disabled by configuration, report that remaining gate.

@coderabbitai

coderabbitai Bot commented Oct 26, 2026

Copy link
Copy Markdown

Tip

For best results, initiate chat on the files or code changes.

@tee8z CodeRabbit approval is disabled because reviews.auto_approvals.enabled is false. I cannot submit an approval for head 4b31928c1ccd3e5c3496d1807ba4dd05afcf0daa.

No approval was submitted. The current GitHub approval status remains unverified. If branch protection requires approval, an authorized reviewer must provide it. All review and pre-merge requirements remain in effect.

You are interacting with an AI system.

@tee8z
tee8z merged commit 02a7cb6 into main Oct 26, 2026
92 checks passed
@tee8z
tee8z deleted the fix/fmt-profile-tests branch October 26, 2026 19:21
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.

2 participants