psa-systems/mokosh-server
Watch
9
0
Fork
You've already forked mokosh-server
 1
Code Issues Pull requests Projects Releases 10 Packages Wiki Activity Actions

style(fmt): apply cargo fmt --all to fix check.yml rustfmt failure #165

Closed
David wants to merge 0 commits from david/fix/pms-214-cargo-fmt into main AGit
pull from: david/fix/pms-214-cargo-fmt
merge into: psa-systems:main
Conversation 1 Commits - Files changed 550 +63 -32
David commented 2026-06-11 14:09:30 +02:00
Owner
Copy link

The TenantId sweep (PMS-139 batches) introduced formatting diffs (line-length wrapping, import ordering) across calendar, contracts, knowledge_base, settings, sla, tenants, and auth middleware. The check.yml job runs rustfmt in --check mode and exited 1. Running cargo fmt --all normalizes the affected files and brings the check suite back to green.

#PMS-214

The TenantId sweep (PMS-139 batches) introduced formatting diffs (line-length wrapping, import ordering) across calendar, contracts, knowledge_base, settings, sla, tenants, and auth middleware. The check.yml job runs rustfmt in --check mode and exited 1. Running cargo fmt --all normalizes the affected files and brings the check suite back to green. #PMS-214
style(fmt): apply cargo fmt --all to fix check.yml rustfmt failure
Some checks failed
E2E (staging) / Playwright against staging (pull_request) Failing after 27s
Details
Check / fmt + clippy + compile + tests (pull_request) Successful in 2m34s
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
675ecf759b 
The TenantId sweep (PMS-139 batches) introduced formatting diffs (line-length wrapping, import ordering) across calendar, contracts, knowledge_base, settings, sla, tenants, and auth middleware. The check.yml job runs rustfmt in --check mode and exited 1. Running cargo fmt --all normalizes the affected files and brings the check suite back to green.

#PMS-214
David commented 2026-06-13 12:39:53 +02:00
Author
Owner
Copy link

Closing as redundant. This PR's only commit was a cargo fmt fixup (PMS-214) for rustfmt diffs introduced by the PMS-139 TenantId sweep. That formatting is already in main: PR #173 (chore: cargo fmt --all after the PMS-139 TenantId sweep, commit c441b1c) reformatted the whole workspace and superseded these narrow fixups. Rebasing this PR's head onto current main drops the commit ("patch contents already upstream"), leaving nothing to merge.

Closing as redundant. This PR's only commit was a `cargo fmt` fixup (PMS-214) for rustfmt diffs introduced by the PMS-139 TenantId sweep. That formatting is already in `main`: PR #173 (`chore: cargo fmt --all after the PMS-139 TenantId sweep`, commit `c441b1c`) reformatted the whole workspace and superseded these narrow fixups. Rebasing this PR's head onto current `main` drops the commit ("patch contents already upstream"), leaving nothing to merge.
David closed this pull request 2026-06-13 12:39:54 +02:00
Some checks are pending
E2E (staging) / Playwright against staging (pull_request) Failing after 27s
Details
Check / fmt + clippy + compile + tests (pull_request) Successful in 2m34s
Required
Details
Create release / Create release from merged PR (pull_request) Has been skipped
Details
E2E / * (pull_request)
Required

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set

Reference
psa-systems/mokosh-server!165
No description provided.