Skip to content

Fix cargo fmt - #979

Merged
louis-e merged 1 commit into
mainfrom
fix-cargo-fmt-main
Apr 24, 2026
Merged

louis-e merged 1 commit into
mainfrom
fix-cargo-fmt-main

Conversation

@louis-e

@louis-e louis-e commented Apr 24, 2026

Copy link
Copy Markdown
Owner

No description provided.

Copilot AI review requested due to automatic review settings April 24, 2026 01:14
@louis-e
louis-e merged commit b333545 into main Apr 24, 2026
4 checks passed
@louis-e
louis-e deleted the fix-cargo-fmt-main branch April 24, 2026 01:15

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This PR applies cargo fmt-driven formatting fixes to Rust source files to align with the project’s standard Rust style and keep diffs consistent across the codebase.

Changes:

  • Reformats array literals and function signatures in element_processing::mod for rustfmt compliance.
  • Cleans up import ordering and removes extra blank lines in amenities.rs.
  • Adds a trailing comma in a multi-line function call argument list in data_processing.rs (rustfmt style).

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
src/element_processing/mod.rs Reflows candidate coordinate list and formats public wrapper function signatures.
src/element_processing/amenities.rs Normalizes use ordering and removes redundant blank lines.
src/data_processing.rs Adds trailing comma to match rustfmt formatting for multi-line calls.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@github-actions

Copy link
Copy Markdown

⏱️ Benchmark run finished in 0m 53s
🏗️ Generation time: 27s (excl. data fetching)
🧠 Peak memory usage: 1859 MB

📈 Compared against baseline: 27s
🧮 Delta: 26s
🔢 Commit: ceeda61

🟢 Generation time is unchanged.

📅 Last benchmark: 2026-04-24 01:17:22 UTC

You can retrigger the benchmark by commenting retrigger-benchmark.

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