Python
/
uv
mirror of https://github.com/astral-sh/uv
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

Clippy

This commit is contained in:
Zanie 2024-01-09 18:25:15 -06:00
parent 67ac01ae75
commit 17901f6438
1 changed files with 8 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

17
crates/puffin-resolver/src/pubgrub/report.rs
View File

@ -1,7 +1,6 @@
use std::borrow::Cow; use std::borrow::Cow;
use std::ops::Bound; use std::ops::Bound;
use chrono::format::Pad;
use derivative::Derivative; use derivative::Derivative;
use owo_colors::OwoColorize; use owo_colors::OwoColorize;
use pubgrub::range::Range; use pubgrub::range::Range;
@ -157,7 +156,7 @@ impl ReportFormatter<PubGrubPackage, Range<PubGrubVersion>> for PubGrubReportFor
slice => { slice => {
let str_terms: Vec<_> = slice let str_terms: Vec<_> = slice
.iter() .iter()
.map(|(p, t)| format!("{p}{}", PackageTerm::new(p, *t))) .map(|(p, t)| format!("{p}{}", PackageTerm::new(p, t)))
.collect(); .collect();
str_terms.join(", ") + " are incompatible" str_terms.join(", ") + " are incompatible"
} }
@ -424,7 +423,7 @@ impl std::fmt::Display for PubGrubHint {
} }
} }
/// A [Term] and [PubGrubPackage] combination for display. /// A [`Term`] and [`PubGrubPackage`] combination for display.
struct PackageTerm<'a> { struct PackageTerm<'a> {
package: &'a PubGrubPackage, package: &'a PubGrubPackage,
term: &'a Term<Range<PubGrubVersion>>, term: &'a Term<Range<PubGrubVersion>>,
@ -454,14 +453,14 @@ impl PackageTerm<'_> {
} }
} }
/// The kind of version ranges being displayed in [PackageRange] /// The kind of version ranges being displayed in [`PackageRange`]
#[derive(Debug)] #[derive(Debug)]
enum PackageRangeKind { enum PackageRangeKind {
Depends, Depends,
Requires, Requires,
} }
/// A [Range] and [PubGrubPackage] combination for display. /// A [`Range`] and [`PubGrubPackage`] combination for display.
#[derive(Debug)] #[derive(Debug)]
struct PackageRange<'a> { struct PackageRange<'a> {
package: &'a PubGrubPackage, package: &'a PubGrubPackage,
@ -481,7 +480,7 @@ impl std::fmt::Display for PackageRange<'_> {
PackageRangeKind::Requires => write!(f, "any of:")?, PackageRangeKind::Requires => write!(f, "any of:")?,
} }
} }
for (_idx, segment) in segments.iter().enumerate() { for segment in &segments {
if segments.len() > 1 { if segments.len() > 1 {
write!(f, "\n ")?; write!(f, "\n ")?;
} }
@ -505,7 +504,7 @@ impl std::fmt::Display for PackageRange<'_> {
}; };
} }
if segments.len() > 1 { if segments.len() > 1 {
write!(f, "\n")?; writeln!(f)?;
} }
} }
Ok(()) Ok(())
@ -570,7 +569,7 @@ impl<T: std::fmt::Display> std::fmt::Display for Padded<'_, T> {
if let Some(char) = content.chars().next() { if let Some(char) = content.chars().next() {
if !char.is_whitespace() { if !char.is_whitespace() {
result.push_str(self.left) result.push_str(self.left);
} }
} }
@ -578,7 +577,7 @@ impl<T: std::fmt::Display> std::fmt::Display for Padded<'_, T> {
if let Some(char) = content.chars().last() { if let Some(char) = content.chars().last() {
if !char.is_whitespace() { if !char.is_whitespace() {
result.push_str(self.right) result.push_str(self.right);
} }
} }