clippy says: fn new should usually return self <-- I picked ignore, since this is test code waiting to be moved. And likely also renamed from "new" to maybe "new_mock_block" to make sure nobody finds it and wants to use it to build a non-test block
@@ -418,6 +418,7 @@ impl Block {
/// TODO - Move this somewhere where only tests will use it.
/// *** Only used in tests. ***
///
+ #[warn(clippy::new_ret_no_self)]
pub fn new(
prev: &BlockHeader,
txs: Vec<Transaction>,
