Skip to content

fix(container): add clippy and rustfmt rustup components#5

Merged
matthew-on-git merged 1 commit intomainfrom
fix/rust-clippy-rustfmt
Mar 5, 2026
Merged

fix(container): add clippy and rustfmt rustup components#5
matthew-on-git merged 1 commit intomainfrom
fix/rust-clippy-rustfmt

Conversation

@matthew-on-git
Copy link
Contributor

@matthew-on-git matthew-on-git commented Mar 5, 2026 •
edited
Loading

Summary

  • Add rustup component add clippy rustfmt to the rust-builder stage in the Dockerfile
  • The rust:1-slim-bookworm base image only ships cargo, rustc, and rust-std — clippy and rustfmt must be explicitly added

Test plan

  • Built container locally, verified cargo clippy --version → 0.1.93
  • Verified rustfmt --version → 1.8.0-stable
  • Version manifest script now reports real versions instead of "unknown"
  • CI passes

🤖 Generated with Claude Code

@matthew-on-git @claude
fix(container): add clippy and rustfmt rustup components
554df2f 
The rust:1-slim-bookworm base image does not ship clippy or rustfmt
by default. Add `rustup component add clippy rustfmt` in the builder
stage so both tools are available in the final image.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@matthew-on-git matthew-on-git merged commit 0ccf41b into main Mar 5, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@matthew-on-git