refactor: lint

- allow `doc_paragraphs_missing_punctuation` globally
- fix `unnecessary_sort_by` in `chain_seeds` (keep descending order via `Reverse`)
- fix `multiple_inherent_impl` in `Concat` by merging impl blocks

Author: ivan-aksamentov

Cargo.toml

@@ -171,6 +171,7 @@ comparison_chain = "allow"
 default_numeric_fallback = "allow"
 deref_by_slicing = "allow"
 doc_markdown = "allow"
+doc_paragraphs_missing_punctuation = "allow"
 else_if_without_else = "allow"
 exhaustive_enums = "allow"
 exhaustive_structs = "allow"

packages/nextclade/src/align/seed_match.rs

@@ -12,7 +12,7 @@ use gcollections::ops::{Intersection, IsEmpty, Union};
 use interval::interval_set::{IntervalSet, ToIntervalSet};
 use itertools::Itertools;
 use std::borrow::Cow;
-use std::cmp::{max, min};
+use std::cmp::{Reverse, max, min};
 use std::collections::{BTreeMap, VecDeque};
 
 /// Copied from https://stackoverflow.com/a/75084739/7483211
@@ -413,7 +413,7 @@ fn chain_seeds(matches: &[SeedMatch2]) -> Vec<SeedMatch2> {
           };
           triplets.push(added_triplet);
           // Sort descending by ref_end
-          triplets.sort_by(|b, a| a.ref_end.cmp(&b.ref_end));
+          triplets.sort_by_key(|triplet| Reverse(triplet.ref_end));
           triplets.retain(|triplet| triplet.ref_end < added_triplet.ref_end || triplet.score >= added_triplet.score);
         }
       }

packages/nextclade/src/io/concat.rs

@@ -46,13 +46,7 @@ where
     let curr = iter.next();
     Concat { iter, curr, delimiter }
   }
-}
 
-impl<I> Concat<I>
-where
-  I: Iterator,
-  <I as Iterator>::Item: Read,
-{
   /// Returns a reference to the item last read, or None if the iterator has been exhausted.
   ///
   /// This is useful for error handling and reporting: if a read operation fails, the reference