Skip to content

Apply rustfmt across Rust sources - #73

Merged
hyperpolymath merged 3 commits into
mainfrom
coderabbit/fix-rust-formatting/0994e5a8
Sep 9, 2026
Merged

Apply rustfmt across Rust sources#73
hyperpolymath merged 3 commits into
mainfrom
coderabbit/fix-rust-formatting/0994e5a8

Conversation

@coderabbitai

@coderabbitai coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

Reformat four Rust source files to satisfy CI formatting checks, including import ordering and standard wrapping of assertions, URLs, iterator chains, and comments. These are formatting-only changes with no intended behavioral impact.

Validation: git diff --check passed; no test suite was run.

View coding task

hyperpolymath and others added 2 commits September 9, 2026 02:19
This repo's standards reusable pins are re-pointed at the standards default-branch HEAD, resolved live at sweep time. The prior refs are recorded in the verification line below. Three kinds of drift are repaired together and the body does not claim which one this repo had: an UNREACHABLE sha kills the run at workflow STARTUP, so GitHub reports no check at all rather than a failing one and the gate disappears instead of going red; a FLOATING ref (@main) is unpinned supply chain; a merely STALE but reachable sha silently reintroduces every bug fixed since it. files=6 pins=6 perms=0 permlines=0 from=092dedada188f56c5915f74a5fd40aac093742c3,5b1d00229e5e8c0c0fbfedc7e80f37ea50f49236 target=8f2ee508

Co-Authored-By: Claude Opus 5 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_0178nN4Nm3neFRy5K9StZKnB
@coderabbitai

coderabbitai Bot commented Sep 9, 2026

Copy link
Copy Markdown
Contributor Author

Review Change StackReview Change Stack

Caution

Review failed

The pull request is closed.

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Organization UI

Review profile: ASSERTIVE

Plan: Essentials

Run ID: 4a4f7b65-8d9a-4a2a-8433-9409264d0b5a

📥 Commits

Reviewing files that changed from the base of the PR and between 485d78e and b02740d.

📒 Files selected for processing (3)
  • crates/squabble-cli/src/fetch.rs
  • crates/squabble-cli/src/fight.rs
  • crates/squabble-core/src/polarity.rs

📝 Summary

Summary by CodeRabbit

  • Style

    • Reformatted internal test URLs, assertions, and conditional expressions for consistency.
    • Updated internal imports without changing functionality.
  • Tests

    • Test coverage and runtime behaviour remain unchanged.

Walkthrough

The pull request applies Rust formatting changes to CLI URLs, assertions, imports, and polarity conditions. Runtime behaviour, public declarations, and test coverage remain unchanged.

Changes

Maintenance updates

Layer / File(s) Summary
Rust formatting updates
crates/squabble-cli/src/fetch.rs, crates/squabble-cli/src/fight.rs, crates/squabble-core/src/polarity.rs
Reformats test URLs, assertions, polarity conditions, and the polarity import without changing behaviour.

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

Suggested reviewers: hyperpolymath


A rabbit checks each tidy line
URLs fold in a neat design
Imports hop into their place
Assertions keep their steady face
No behaviours change or stray
Clean Rust bounds along its way

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

Base automatically changed from fix/bump-diverged-standards-pin to main September 9, 2026 09:36
hyperpolymath
hyperpolymath previously approved these changes Sep 9, 2026
Signed-off-by: Jonathan D.A. Jewell <6759885+hyperpolymath@users.noreply.github.com>
@sonarqubecloud

sonarqubecloud Bot commented Sep 9, 2026

Copy link
Copy Markdown

@hyperpolymath
hyperpolymath merged commit 327fb24 into main Sep 9, 2026
46 of 49 checks passed
@hyperpolymath
hyperpolymath deleted the coderabbit/fix-rust-formatting/0994e5a8 branch September 9, 2026 16:00
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.

1 participant