Skip to content

Avoid excessive Orchard dummies in `TestState::generate_next_block_sp… #4836

Avoid excessive Orchard dummies in `TestState::generate_next_block_sp…

Avoid excessive Orchard dummies in `TestState::generate_next_block_sp… #4836

Triggered via push March 8, 2024 01:39
Status Failure
Total duration 29m 48s
Artifacts

ci.yml

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

Annotations

6 errors and 5 warnings
Clippy (beta)
The process '/home/runner/.cargo/bin/cargo' failed with exit code 1
this function has too many arguments (9/7): zcash_client_sqlite/src/testing.rs#L795
error: this function has too many arguments (9/7) --> zcash_client_sqlite/src/testing.rs:795:5 | 795 | / fn add_compact_action<P: consensus::Parameters, R: RngCore + CryptoRng>( 796 | | &self, 797 | | ctx: &mut CompactTx, 798 | | params: &P, ... | 804 | | rng: &mut R, 805 | | ) -> Self::Nullifier { | |________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
this function has too many arguments (8/7): zcash_client_sqlite/src/testing.rs#L784
error: this function has too many arguments (8/7) --> zcash_client_sqlite/src/testing.rs:784:5 | 784 | / fn add_compact_output<P: consensus::Parameters, R: RngCore + CryptoRng>( 785 | | &self, 786 | | ctx: &mut CompactTx, 787 | | params: &P, ... | 792 | | rng: &mut R, 793 | | ) -> Self::Nullifier; | |________________________^ | = note: `-D clippy::too-many-arguments` implied by `-D warnings` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
Clippy (MSRV)
Clippy had exited with the 101 exit code
this function has too many arguments (9/7): zcash_client_sqlite/src/testing.rs#L795
error: this function has too many arguments (9/7) --> zcash_client_sqlite/src/testing.rs:795:5 | 795 | / fn add_compact_action<P: consensus::Parameters, R: RngCore + CryptoRng>( 796 | | &self, 797 | | ctx: &mut CompactTx, 798 | | params: &P, ... | 804 | | rng: &mut R, 805 | | ) -> Self::Nullifier { | |________________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
this function has too many arguments (8/7): zcash_client_sqlite/src/testing.rs#L784
error: this function has too many arguments (8/7) --> zcash_client_sqlite/src/testing.rs:784:5 | 784 | / fn add_compact_output<P: consensus::Parameters, R: RngCore + CryptoRng>( 785 | | &self, 786 | | ctx: &mut CompactTx, 787 | | params: &P, ... | 792 | | rng: &mut R, 793 | | ) -> Self::Nullifier; | |________________________^ | = note: `-D clippy::too-many-arguments` implied by `-D warnings` = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments
Clippy (beta)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions-rs/clippy-check@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Clippy (beta)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
protobuf consistency
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: supplypike/setup-bin@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Clippy (MSRV)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions-rs/clippy-check@v1. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Clippy (MSRV)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions-rs/clippy-check@v1. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/