Skip to content

orchard 0.6.0

orchard 0.6.0 #3030

Triggered via push September 8, 2023 17:52
Status Success
Total duration 31m 19s
Artifacts

ci.yml

on: push
Matrix: build
Matrix: test
Fit to window
Zoom out
Zoom in

Annotations

4 errors and 36 warnings
incorrect implementation of `partial_cmp` on an `Ord` type: src/primitives/redpallas.rs#L102
error: incorrect implementation of `partial_cmp` on an `Ord` type --> src/primitives/redpallas.rs:102:1 | 102 | / impl<T: SigType> PartialOrd for VerificationKey<T> { 103 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 104 | || <[u8; 32]>::from(self).partial_cmp(&<[u8; 32]>::from(other)) 105 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 106 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incorrect_partial_ord_impl_on_ord_type = note: `#[deny(clippy::incorrect_partial_ord_impl_on_ord_type)]` on by default
incorrect implementation of `partial_cmp` on an `Ord` type: src/primitives/redpallas.rs#L102
error: incorrect implementation of `partial_cmp` on an `Ord` type --> src/primitives/redpallas.rs:102:1 | 102 | / impl<T: SigType> PartialOrd for VerificationKey<T> { 103 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 104 | || <[u8; 32]>::from(self).partial_cmp(&<[u8; 32]>::from(other)) 105 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 106 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incorrect_partial_ord_impl_on_ord_type = note: `#[deny(clippy::incorrect_partial_ord_impl_on_ord_type)]` on by default
incorrect implementation of `partial_cmp` on an `Ord` type: src/primitives/redpallas.rs#L102
error: incorrect implementation of `partial_cmp` on an `Ord` type --> src/primitives/redpallas.rs:102:1 | 102 | / impl<T: SigType> PartialOrd for VerificationKey<T> { 103 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 104 | || <[u8; 32]>::from(self).partial_cmp(&<[u8; 32]>::from(other)) 105 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 106 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incorrect_partial_ord_impl_on_ord_type = note: `#[deny(clippy::incorrect_partial_ord_impl_on_ord_type)]` on by default
incorrect implementation of `partial_cmp` on an `Ord` type: src/primitives/redpallas.rs#L102
error: incorrect implementation of `partial_cmp` on an `Ord` type --> src/primitives/redpallas.rs:102:1 | 102 | / impl<T: SigType> PartialOrd for VerificationKey<T> { 103 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 104 | || <[u8; 32]>::from(self).partial_cmp(&<[u8; 32]>::from(other)) 105 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 106 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#incorrect_partial_ord_impl_on_ord_type = note: `#[deny(clippy::incorrect_partial_ord_impl_on_ord_type)]` on by default
Book tests
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Rustfmt
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L201
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:201:14 | 201 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L195
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:195:14 | 195 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + *v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/note_encryption.rs#L246
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/note_encryption.rs:246:18 | 246 | .zip(ephemeral_keys.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `ephemeral_keys` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L201
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:201:14 | 201 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L195
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:195:14 | 195 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + *v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/note_encryption.rs#L246
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/note_encryption.rs:246:18 | 246 | .zip(ephemeral_keys.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `ephemeral_keys` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L430
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:430:14 | 430 | .fold(Some(ValueSum::zero()), |acc, action| { | ______________^ 431 | | acc? + action.value_sum() 432 | | }) | |______________^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, action| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/builder.rs#L418
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:418:22 | 418 | .zip(self.recipients.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `self.recipients` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L379
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:379:14 | 379 | .fold(Some(ValueSum::zero()), |acc, note_value| acc? + note_value) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, note_value| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L430
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:430:14 | 430 | .fold(Some(ValueSum::zero()), |acc, action| { | ______________^ 431 | | acc? + action.value_sum() 432 | | }) | |______________^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, action| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/builder.rs#L418
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:418:22 | 418 | .zip(self.recipients.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `self.recipients` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L379
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:379:14 | 379 | .fold(Some(ValueSum::zero()), |acc, note_value| acc? + note_value) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, note_value| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default
Intra-doc links
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Bitrot check
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test on ubuntu-latest
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test on macOS-latest
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Test on windows-latest
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Code coverage
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/checkout@v2, actions-rs/toolchain@v1, actions-rs/cargo@v1, codecov/codecov-action@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
Code coverage
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Code coverage
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Code coverage
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
Code coverage
The `set-output` command is deprecated and will be disabled soon. Please upgrade to using Environment Files. For more information see: https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L201
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:201:14 | 201 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L195
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:195:14 | 195 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + *v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/note_encryption.rs#L246
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/note_encryption.rs:246:18 | 246 | .zip(ephemeral_keys.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `ephemeral_keys` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L201
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:201:14 | 201 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
usage of `Iterator::fold` on a type that implements `Try`: src/value.rs#L195
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/value.rs:195:14 | 195 | iter.fold(Ok(ValueSum(0)), |acc, v| (acc? + *v).ok_or(OverflowError)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum(0), |acc, v| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/note_encryption.rs#L246
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/note_encryption.rs:246:18 | 246 | .zip(ephemeral_keys.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `ephemeral_keys` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L430
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:430:14 | 430 | .fold(Some(ValueSum::zero()), |acc, action| { | ______________^ 431 | | acc? + action.value_sum() 432 | | }) | |______________^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, action| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/builder.rs#L418
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:418:22 | 418 | .zip(self.recipients.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `self.recipients` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L379
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:379:14 | 379 | .fold(Some(ValueSum::zero()), |acc, note_value| acc? + note_value) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, note_value| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L430
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:430:14 | 430 | .fold(Some(ValueSum::zero()), |acc, action| { | ______________^ 431 | | acc? + action.value_sum() 432 | | }) | |______________^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, action| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold
explicit call to `.into_iter()` in function argument accepting `IntoIterator`: src/builder.rs#L418
warning: explicit call to `.into_iter()` in function argument accepting `IntoIterator` --> src/builder.rs:418:22 | 418 | .zip(self.recipients.into_iter()) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: consider removing the `.into_iter()`: `self.recipients` | note: this parameter accepts any `IntoIterator`, so you don't need to call `.into_iter()` --> /rustc/9f37cd4f79d289492ccaa8f28733b765498c95e5/library/core/src/iter/traits/iterator.rs:642:12 = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#useless_conversion = note: `#[warn(clippy::useless_conversion)]` on by default
usage of `Iterator::fold` on a type that implements `Try`: src/builder.rs#L379
warning: usage of `Iterator::fold` on a type that implements `Try` --> src/builder.rs:379:14 | 379 | .fold(Some(ValueSum::zero()), |acc, note_value| acc? + note_value) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: use `try_fold` instead: `try_fold(ValueSum::zero(), |acc, note_value| ...)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#manual_try_fold = note: `#[warn(clippy::manual_try_fold)]` on by default