Skip to content

Commit

Permalink
Clippy lints update: all is implied, add todo (#2799)
Browse files Browse the repository at this point in the history
I must have been mistaken when originally introducing `clippy:all`, because these are, by definition, lints that are enabled without any configuration: https://doc.rust-lang.org/stable/clippy/usage.html#lint-configuration

Add a lint that catches `todo!()` (but allows `panic!()`, `unreachable!()`, `unimplemented!()`) - you're encouraged to use it during development as a reminder for stuff to finish before a change can be merged.

Sibling in `actor`: tonarino/actor#78
  • Loading branch information
strohel authored Oct 20, 2023
1 parent 07f88af commit 6f19422
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/lib.rs
Original file line number Diff line number Diff line change
@@ -1,4 +1,4 @@
#![warn(clippy::all, clippy::clone_on_ref_ptr, clippy::mod_module_files)]
#![warn(clippy::clone_on_ref_ptr, clippy::mod_module_files, clippy::todo)]

use image::{buffer::ConvertBuffer, open, Bgr, ImageBuffer};
use serde::Serialize;
Expand Down

0 comments on commit 6f19422

Please sign in to comment.