style: apply cargo fmt across src and tests

David commented

2026-06-11 14:29:50 +02:00

Owner

The check.yml CI job failed on cargo fmt --check (run #813), printing formatting diffs across src/ and tests/ files: import reordering (uuid::Uuid vs crate::modules::auth::TenantId) and call-site line-wrapping introduced by the PMS-139 TenantId sweep. Run cargo fmt --all to bring the tree back to canonical rustfmt output. No functional changes.

#PMS-226

The check.yml CI job failed on cargo fmt --check (run #813), printing formatting diffs across src/ and tests/ files: import reordering (uuid::Uuid vs crate::modules::auth::TenantId) and call-site line-wrapping introduced by the PMS-139 TenantId sweep. Run cargo fmt --all to bring the tree back to canonical rustfmt output. No functional changes. #PMS-226

David added 1 commit

2026-06-11 14:29:50 +02:00

style: apply cargo fmt across src and tests

E2E (staging) / Playwright against staging (pull_request) Failing after 28s

Details

Check / fmt + clippy + compile + tests (pull_request) Successful in 2m25s

Details

Create release / Create release from merged PR (pull_request) Has been skipped

Details

728bb1f441

The check.yml CI job failed on cargo fmt --check (run #813), printing formatting diffs across src/ and tests/ files: import reordering (uuid::Uuid vs crate::modules::auth::TenantId) and call-site line-wrapping introduced by the PMS-139 TenantId sweep. Run cargo fmt --all to bring the tree back to canonical rustfmt output. No functional changes.

#PMS-226

David commented

2026-06-13 12:41:32 +02:00

Author

Owner

Closing as redundant and stale. This PR's only commit was a cargo fmt fixup (PMS-226). Two things have since happened in main:

The formatting intent is already served: PR #173 (chore: cargo fmt --all after the PMS-139 TenantId sweep, c441b1c) and PR #183 (chore(fmt): apply cargo fmt --all after PMS-244..PMS-249 sweep) reformatted the whole workspace.

The specific lines this PR reflows are obsolete. It wraps TenantId::from_trusted(tenant_id) at the tickets notify/dispatch call sites, a transitional form. The tickets hub was fully migrated to TenantId in PR #171 (sweep batch 8), so main now passes the already-typed tenant_id directly. That divergence is the source of the merge conflict (in src/modules/tickets/automation.rs and src/modules/tickets/service.rs); resolving toward main loses nothing this PR intended, since it intended only formatting.

It is an AGit PR with no server branch (head exists only at refs/pull/170/head), so "Update Branch" was never available. Nothing left to merge.

Closing as redundant and stale. This PR's only commit was a `cargo fmt` fixup (PMS-226). Two things have since happened in `main`: The formatting intent is already served: PR #173 (`chore: cargo fmt --all after the PMS-139 TenantId sweep`, `c441b1c`) and PR #183 (`chore(fmt): apply cargo fmt --all after PMS-244..PMS-249 sweep`) reformatted the whole workspace. The specific lines this PR reflows are obsolete. It wraps `TenantId::from_trusted(tenant_id)` at the tickets notify/dispatch call sites, a transitional form. The tickets hub was fully migrated to `TenantId` in PR #171 (sweep batch 8), so `main` now passes the already-typed `tenant_id` directly. That divergence is the source of the merge conflict (in `src/modules/tickets/automation.rs` and `src/modules/tickets/service.rs`); resolving toward `main` loses nothing this PR intended, since it intended only formatting. It is an AGit PR with no server branch (head exists only at `refs/pull/170/head`), so "Update Branch" was never available. Nothing left to merge.