Skip to content

fmt: rustfmt arigraph/community.rs (unblocks main CI after #714)#715

Merged
AdaWorldAPI merged 1 commit into
mainfrom
claude/review-claude-board-files-nhqgx1
Jul 17, 2026
Merged

fmt: rustfmt arigraph/community.rs (unblocks main CI after #714)#715
AdaWorldAPI merged 1 commit into
mainfrom
claude/review-claude-board-files-nhqgx1

Conversation

@AdaWorldAPI

@AdaWorldAPI AdaWorldAPI commented Jul 17, 2026

Copy link
Copy Markdown
Owner

What

#714 (feat(arigraph): community.rs, D-GR-3a) merged without cargo fmt, so cargo fmt --manifest-path crates/lance-graph/Cargo.toml -- --check now fails on main and blocks CI on every subsequent PR.

This applies cargo fmt to that one file. Pure formatting — zero semantic change (block-body vs single-line closures, struct-literal field wrapping, test tuple wrapping — exactly the diff the CI job printed).

Verification

  • cargo fmt --manifest-path crates/lance-graph/Cargo.toml -- --checkclean.
  • git diff --stat → single file (community.rs, 61+/21−); nothing else touched, confirming main is otherwise fmt-clean.

🤖 Generated with Claude Code

https://claude.ai/code/session_01Awg6TXocHcwTtc6eGsHcdD


Generated by Claude Code

Summary by CodeRabbit

  • Style
    • Reformatted graph community detection code for improved readability.
    • Rewrapped related test data and assertions without changing behavior or expected results.

#714 (feat(arigraph): community.rs, D-GR-3a) merged without `cargo fmt`, so
`cargo fmt --manifest-path crates/lance-graph/Cargo.toml -- --check` fails on
main and blocks CI on every subsequent PR. Pure formatting — zero semantic
change (block-body vs single-line closures, struct-literal wrapping, test
tuple wrapping).

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01Awg6TXocHcwTtc6eGsHcdD
@cursor

cursor Bot commented Jul 17, 2026

Copy link
Copy Markdown

Bugbot couldn't run - usage limit reached

Bugbot is counted against Cursor usage for this user or team, and this run hit a usage or spend limit.

A user or team admin can review and increase usage limits in the Cursor dashboard.

(requestId: serverGenReqId_91fdbd4c-dea3-4932-85a0-c782087e6cf7)

@coderabbitai

coderabbitai Bot commented Jul 17, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 356087d5-e04d-4301-aa6f-c0943dd47411

📥 Commits

Reviewing files that changed from the base of the PR and between 79ff75e and f7e01a0.

📒 Files selected for processing (1)
  • crates/lance-graph/src/graph/arigraph/community.rs

📝 Walkthrough

Walkthrough

Formatting-only updates rewrap community detection implementation and test code in community.rs; logic, public APIs, test data, assertions, and outcomes remain unchanged.

Changes

Community Detection Formatting

Layer / File(s) Summary
Reformat implementation and tests
crates/lance-graph/src/graph/arigraph/community.rs
Rewrapped community membership, graph construction, adjacency aggregation, function signatures, test edge lists, and truth-value construction without changing behavior or assertions.

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

Suggested reviewers: claude

Poem

A rabbit hops through neatly spaced code,
No logic changed along the road.
Graphs and tests now line up bright,
Every brace sits just right.
“Format!” I cheer, then nibble a byte.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title accurately summarizes the formatting-only change to community.rs and the CI fix context.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
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
  • Create stacked PR
  • Commit on current branch

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.

@AdaWorldAPI
AdaWorldAPI merged commit e9e00be into main Jul 17, 2026
7 checks passed
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