Skip to content

style(app): apply rustfmt to satisfy CI format check (#1954) - #1955

Merged
datlechin merged 1 commit into
TableProApp:linuxfrom
Seha16:fix/cargo-fmt
Jul 25, 2026
Merged

style(app): apply rustfmt to satisfy CI format check (#1954)#1955
datlechin merged 1 commit into
TableProApp:linuxfrom
Seha16:fix/cargo-fmt

Conversation

@Seha16

@Seha16 Seha16 commented Jul 25, 2026

Copy link
Copy Markdown
Contributor

Summary

  • The linux branch fails cargo fmt --all -- --check under the pinned 1.93 toolchain (rustfmt 1.8.0), turning the Fast checks CI job red before build / clippy / tests run.
  • This runs cargo fmt --all over the two drifted files (crates/app/src/ui/app/mod.rs, crates/app/src/ui/browse_tab.rs): a stray blank line and a few builder chains collapsed to fit max_width = 120.
  • Formatting only, no behavior change (2 files, +4 / -12). Fixes Linux: cargo fmt --all -- --check fails on linux branch (Fast checks CI red) #1954.

Test plan

  • cargo fmt --all -- --check (clean)
  • cargo clippy --all-targets -- -D warnings (clean)
  • cargo build --workspace (exit 0)
  • cargo test --workspace --lib (148 passed, 1 ignored)

No UI/visual change (formatting only), so no before/after screenshots apply.

@datlechin datlechin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Thank you

@datlechin
datlechin merged commit b1a4972 into TableProApp:linux Jul 25, 2026
2 checks passed
@Seha16
Seha16 deleted the fix/cargo-fmt branch July 25, 2026 09:43
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