Skip to content

build: format all file kinds (TOML + Markdown + Rust formatting) with a single tool, dprint#158

Merged
LukeMathWalker merged 2 commits into
Firma-AI:mainfrom
LukeMathWalker:dprint-formatting
Jun 16, 2026
Merged

build: format all file kinds (TOML + Markdown + Rust formatting) with a single tool, dprint#158
LukeMathWalker merged 2 commits into
Firma-AI:mainfrom
LukeMathWalker:dprint-formatting

Conversation

@LukeMathWalker

@LukeMathWalker LukeMathWalker commented Jun 16, 2026

Copy link
Copy Markdown
Collaborator

Use dprint as the single entrypoint for formatting all file kinds (Rust, Markdown, YAML) in the repository.
Split into two revisions for ease of review:

  1. Wire dprint
  2. Apply formatting.

One Rust file (fuzz/fuzz_targets/paseto_verify.rs) was unformatted because fuzz/ is a separate workspace that wasn't covered by the existing fmt CI job.

@LukeMathWalker LukeMathWalker marked this pull request as draft June 16, 2026 09:32
@LukeMathWalker LukeMathWalker force-pushed the dprint-formatting branch 2 times, most recently from 8810a2d to 4d76ba6 Compare June 16, 2026 09:36
dprint becomes the single formatter for the repo, expanding coverage from
Cargo-manifest TOML only to all TOML, Markdown, and Rust under one tool and
one CI job. TOML behavior matches taplo: dependency tables are sorted by
dprint's default cargo conventions.

- add dprint.json (toml, markdown, and exec/rustfmt plugins pinned by
  version + sha256 checksum)
- Makefile: fmt now runs 'dprint check' (covers all files); install dprint
- ci.yml: single Format job uses dprint/check@v2.3 + rustfmt toolchain;
  drops the taplo curl install and the separate cargo fmt job
- CLAUDE.md: document the dprint workflow
- remove now-unused .taplo.toml

Formatting is applied in the following revision.
@LukeMathWalker LukeMathWalker marked this pull request as ready for review June 16, 2026 09:41
@LukeMathWalker LukeMathWalker changed the title build: replace taplo with dprint (TOML + Markdown + Rust formatting) build: format all file kinds (TOML + Markdown + Rust formatting) with a single tool, dprint Jun 16, 2026
@LukeMathWalker LukeMathWalker merged commit caaf90c into Firma-AI:main Jun 16, 2026
12 checks passed
@LukeMathWalker LukeMathWalker deleted the dprint-formatting branch June 16, 2026 11:50
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