Skip to content

feat: format build.zig#2

Merged
wemeetagain merged 2 commits intomainfrom
feat/fmt-build-zig
Jul 16, 2025
Merged

feat: format build.zig#2
wemeetagain merged 2 commits intomainfrom
feat/fmt-build-zig

Conversation

@wemeetagain
Copy link
Copy Markdown
Member

@wemeetagain wemeetagain commented Jul 16, 2025

Before, any feature here that alters codegen can cause diffs in generated build.zig due to formatting errors. This is quite annoying. (see ChainSafe/ssz-z#42 as an example)

Now, after syncing build.zig, format it using zig fmt.

Comment thread build.zig Outdated
@wemeetagain wemeetagain merged commit ffb5813 into main Jul 16, 2025
wemeetagain added a commit that referenced this pull request Mar 17, 2026
- #1: Clean stale build.zig.zon (remove deleted exe/test refs, bump to 0.3.0)
- #2: Remove @ptrCast for dest_sub_path (Zig coerces comptime strings)
- #3: Default modules to public (export to b.modules unless private = true)
- #4: @CompileError for unknown option types + validateManifest for unknown
  top-level fields
- #5: resolveImport returns error.ModuleNotFound instead of @Panic
- #6: Remove duplicate modules.put (createModule handles it, callers don't)
- #7: Add 8 comptime helper tests (toStringSlice, toEnumSlice, isIntType,
  isFloatType, isKnownField, validateManifest)

Co-Authored-By: Claude Opus 4.6 <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