Skip to content

ZIR-000: Auto-fix linting and style issues - #346

Closed
zeroecco wants to merge 1 commit into
mainfrom
lint-fix/auto-style-fixes
Closed

zeroecco wants to merge 1 commit into
mainfrom
lint-fix/auto-style-fixes

Conversation

@zeroecco

Copy link
Copy Markdown

Summary

  • Run cargo fmt --all to fix Rust formatting across the workspace
  • Run cargo sort --workspace to sort Cargo.toml dependency ordering (fib crate)
  • Run clang-format -i on all .cpp and .h files in risc0/ and zirgen/ directories
  • License headers verified clean (exit 0, no changes needed)

All changes are purely mechanical output from the project's own tooling — no manual style edits.

Test plan

  • cargo fmt --all -- --check passes (no remaining formatting issues)
  • cargo sort --workspace --check passes
  • python3 license-check.py exits 0

🤖 Generated with Claude Code

Nightshift-Task: lint-fix
Nightshift-Ref: https://github.com/marcus/nightshift

…lang-format, and missing EOL

- cargo fmt --all: Rust source formatting
- cargo sort --workspace: Cargo.toml dependency ordering (zirgen/circuit/fib/Cargo.toml)
- clang-format -i: C++ style fixes across zirgen/
- zirgen/dsl/src/adder.rs: add missing trailing newline (was 0 bytes)

Nightshift-Task: lint-fix
Nightshift-Ref: https://github.com/marcus/nightshift

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@zeroecco
zeroecco force-pushed the lint-fix/auto-style-fixes branch from 26a8899 to cb7af3e Compare May 30, 2026 09:06
@zeroecco

zeroecco commented Jun 6, 2026

Copy link
Copy Markdown
Author

Closing as a duplicate. PR #344 (fix/cargo-sort-fib) is the focused, minimal fix for the cargo-sort violation — it changes only zirgen/circuit/fib/Cargo.toml. The C++ formatting changes in this PR are out of scope for the CI-checked formatters.

@zeroecco zeroecco closed this Jun 6, 2026
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