Skip to content

Add Clippy CI gate#6

Merged
rmorse merged 1 commit into
developfrom
chore/clippy-ci
Jun 13, 2026
Merged

Add Clippy CI gate#6
rmorse merged 1 commit into
developfrom
chore/clippy-ci

Conversation

@rmorse

@rmorse rmorse commented Jun 13, 2026

Copy link
Copy Markdown
Member

Summary

  • Add Clippy to the macOS CI workflow with -D warnings gates for workspace, async, and Metal builds.
  • Clean the current warning baseline across tests, examples, benches, xtask, and the sys build script.
  • Update contributing and publishing docs so the documented checks match CI.

Notes

  • The lint setup stays conservative: default Clippy lints plus -D warnings, not clippy::pedantic.
  • The old compile_fail_tests cfg stub was removed because it was commented-out dead code. If we want compile-fail coverage later, we should add real trybuild tests.

Local verification

  • git diff --check
  • cargo fmt --all -- --check
  • cargo clippy --workspace --all-targets -- -D warnings
  • cargo clippy -p whisper-cpp-plus --all-targets --features async -- -D warnings
  • cargo test --workspace -- --test-threads=1
  • cargo test -p whisper-cpp-plus --features async -- --test-threads=1

Metal Clippy is wired into macOS CI and will be verified there.

@rmorse rmorse merged commit 230b25d into develop Jun 13, 2026
2 checks passed
@rmorse rmorse deleted the chore/clippy-ci branch June 13, 2026 13:45
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