Skip to content

zig fmt: three files main's fmt check refuses - #616

Merged
ashhart merged 1 commit into
ashhart:mainfrom
BobClawblaw:fmt-main
Oct 10, 2026
Merged

ashhart merged 1 commit into
ashhart:mainfrom
BobClawblaw:fmt-main

Conversation

@BobClawblaw

Copy link
Copy Markdown
Contributor

CI's fmt job (zig fmt --check build.zig zig) fails on current main in three files, so it fails on every open PR too (#598 among them):

  • zig/src/core/tokenizer/tokenizer.zig: five test struct literals with several fields to a line
  • zig/src/native/cuda.zig: one line's spacing
  • zig/tests/moe_bench.zig: one blank line

This is zig fmt (0.17.0) on those files and nothing else. Every change is whitespace: each file's bytes without spaces, tabs and newlines are identical before and after. zig fmt --check build.zig zig is clean afterwards.

@ashhart
ashhart merged commit 60aa6ef into ashhart:main Oct 10, 2026
2 checks passed
@ashhart

ashhart commented Oct 10, 2026

Copy link
Copy Markdown
Owner

Thank you, @BobClawblaw. This is merged into main (60aa6ef), your commit as it was, so main passes zig fmt --check build.zig zig again. #591 had changed the CUDA startup line in native/cuda.zig since your base, so the merge keeps main's line as zig fmt writes it.

@ashhart ashhart mentioned this pull request Oct 10, 2026
cwschroeder added a commit to cwschroeder/TensorFold that referenced this pull request Oct 11, 2026
The fmt check refused tokenizer.zig, lane_host.zig and cuda.zig. This PR
changes none of them; they come unformatted from the base 1f938b1, and
upstream formats tokenizer.zig and cuda.zig in ashhart#616. Formatted with
Zig 0.17.0, the version the workflow uses.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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