Skip to content

[skip ci] Apply canonical formatting and repair observed lint findings - #140

Merged
szmyty merged 1 commit into
mainfrom
codex/137-04-format-lint-20261113
Nov 14, 2026
Merged

szmyty merged 1 commit into
mainfrom
codex/137-04-format-lint-20261113

Conversation

@szmyty

@szmyty szmyty commented Nov 13, 2026

Copy link
Copy Markdown
Contributor

Relates to #137, checkpoint 4. Stacked after #139.

Applies the owning canonical formatter (Rustfmt/Cargo 1.85.0) across the declared Cargo targets, pins local Task/CI format commands, and preserves strict compiler lint lanes and the declared MSRV.

Repairs all 39 recorded moving-stable assert_is_empty findings using the historical Clippy typed assert_eq!/assert_ne! suggestions. Also corrects a pre-existing semicolon/comma syntax error in the delivery match arm that prevented the formatter from parsing the module.

Validation: the owning explicit-version cargo fmt --all -- --check commands exited 0; docs/work/137/04-format-lint.json records exact formatter identities and scope. No product type-check/Clippy/build/test, corpus/schema/native or hosted CI execution occurred. Formatting parses source but cannot establish Rust type/borrow correctness or a green Actions suite. No blanket lint allowances or continue-on-error changes.

The extensive diff reflects accumulated deferred formatting; review source layout separately from the named logical repairs (GitHub's whitespace-hidden view can help). No unrelated Identity workspace was formatted, and no fixture expectations are regenerated here. Full compiler/lint acceptance remains open under #137/#88. See docs/work/137/04-format-lint.md. [skip ci]

@szmyty
szmyty changed the base branch from codex/137-03-lock-metadata-20261113 to main November 14, 2026 01:39
@szmyty
szmyty marked this pull request as ready for review November 14, 2026 01:39
@szmyty
szmyty merged commit 3d9a0ac into main Nov 14, 2026
@devactivity-app

Copy link
Copy Markdown

Pull Request Summary by devActivity

Metrics

Cycle Time: < 1 min

Achievements

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.

1 participant