Skip to content

Resolve CI formatting failure in ld2450d socket JSON mapping#2

Merged
duramson merged 2 commits into
mainfrom
copilot/fix-ci-run-failure
Apr 18, 2026
Merged

Resolve CI formatting failure in ld2450d socket JSON mapping#2
duramson merged 2 commits into
mainfrom
copilot/fix-ci-run-failure

Conversation

Copilot AI commented Apr 15, 2026

Copy link
Copy Markdown
Contributor

CI was failing on cargo fmt --check due to non-rustfmt-compliant field alignment in crates/ld2450d/src/socket.rs. This PR applies the minimal formatting-only update required to restore formatter compliance.

  • Formatting cleanup (crates/ld2450d/src/socket.rs)
    • Normalized spacing in TargetJson field declarations.
    • Normalized spacing in TargetJson construction inside frame_to_json.
struct TargetJson {
    x: f32,
    y: f32,
    speed: f32,
    dist: f32,
    angle: f32,
}

.map(|t| TargetJson {
    x: t.x_m(),
    y: t.y_m(),
    speed: t.speed_ms(),
    dist: t.dist_m(),
    angle: t.angle_deg(),
})

Copilot AI linked an issue Apr 15, 2026 that may be closed by this pull request
Copilot AI changed the title [WIP] Fix CI run failure by formatting code Resolve CI formatting failure in ld2450d socket JSON mapping Apr 15, 2026
Copilot AI requested a review from duramson April 15, 2026 17:32

@duramson duramson left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice bro

@duramson duramson marked this pull request as ready for review April 18, 2026 07:33
@duramson duramson merged commit bb4cb29 into main Apr 18, 2026
4 checks passed
duramson added a commit that referenced this pull request Apr 21, 2026
Resolve CI formatting failure in `ld2450d` socket JSON mapping
@duramson duramson deleted the copilot/fix-ci-run-failure branch April 21, 2026 09:07
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.

CI run is failing

2 participants