Skip to content

Can have test generate all files needed #3

Can have test generate all files needed

Can have test generate all files needed #3

Triggered via push August 5, 2024 12:55
Status Failure
Total duration 54s
Artifacts

ci.yml

on: push
Matrix: Rust project - latest
Fit to window
Zoom out
Zoom in

Annotations

5 errors and 6 warnings
Rust project - latest (stable)
Process completed with exit code 101.
Rust project - latest (nightly)
The job was canceled because "stable" failed.
Rust project - latest (nightly)
Process completed with exit code 101.
Rust project - latest (beta)
The job was canceled because "stable" failed.
Rust project - latest (beta)
The operation was canceled.
non-canonical implementation of `partial_cmp` on an `Ord` type: src/distances.rs#L214
warning: non-canonical implementation of `partial_cmp` on an `Ord` type --> src/distances.rs:214:1 | 214 | / impl PartialOrd for SparseCoreAcc { 215 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 216 | || self.1.partial_cmp(&other.1) 217 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 218 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl
non-canonical implementation of `partial_cmp` on an `Ord` type: src/distances.rs#L188
warning: non-canonical implementation of `partial_cmp` on an `Ord` type --> src/distances.rs:188:1 | 188 | / impl PartialOrd for SparseJaccard { 189 | | fn partial_cmp(&self, other: &Self) -> Option<Ordering> { | | _____________________________________________________________- 190 | || other.1.partial_cmp(&self.1) // NB: backwards 191 | || } | ||_____- help: change this to: `{ Some(self.cmp(other)) }` 192 | | } | |__^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#non_canonical_partial_ord_impl = note: `#[warn(clippy::non_canonical_partial_ord_impl)]` on by default
this function has too many arguments (9/7): src/sketch.rs#L205
warning: this function has too many arguments (9/7) --> src/sketch.rs:205:1 | 205 | / pub fn sketch_files( 206 | | output_prefix: &str, 207 | | input_files: &[InputFastx], 208 | | concat_fasta: bool, ... | 214 | | min_qual: u8, 215 | | ) -> Vec<Sketch> { | |________________^ | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#too_many_arguments = note: `#[warn(clippy::too_many_arguments)]` on by default
fields `bin_stride`, `kmer_stride`, and `sample_stride` are never read: src/sketch_datafile.rs#L11
warning: fields `bin_stride`, `kmer_stride`, and `sample_stride` are never read --> src/sketch_datafile.rs:11:5 | 10 | pub struct SketchArrayFile { | --------------- fields in this struct 11 | bin_stride: usize, | ^^^^^^^^^^ 12 | kmer_stride: usize, | ^^^^^^^^^^^ 13 | sample_stride: usize, | ^^^^^^^^^^^^^ | = note: `SketchArrayFile` has a derived impl for the trait `Debug`, but this is intentionally ignored during dead code analysis = note: `#[warn(dead_code)]` on by default
unused variable: `ids`: src/multisketch.rs#L150
warning: unused variable: `ids` --> src/multisketch.rs:150:35 | 150 | pub fn remove_sketches(&self, ids: &[String]) { | ^^^ help: if this is intentional, prefix it with an underscore: `_ids` | = note: `#[warn(unused_variables)]` on by default
Rust project - latest (stable)
The following actions uses Node.js version which is deprecated and will be forced to run on node20: actions/checkout@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/