History

[utils #154] thanks clippy

Sebastian Thiel committed 5 years ago

Commit e16c4821 parent c9a52b91 file · +3 −3Browse files

Unified · Split

etc/utils/src/command/release.rs+3−3
@@ -75,7 +75,7 @@ fn workspace_package_by_name<'a>(meta: &'a Metadata, crate_name: &str) -> Option · 74 unmodified lines
75fn package_by_name<'a>(meta: &'a Metadata, name: &str) -> anyhow::Result<&'a Package> {75fn package_by_name<'a>(meta: &'a Metadata, name: &str) -> anyhow::Result<&'a Package> {
76 meta.packages76 meta.packages
77 .iter()77 .iter()
78 .find(|p| &p.name == name)78 .find(|p| p.name == name)
79 .ok_or_else(|| anyhow!("workspace member must be a listed package: '{}'", name))79 .ok_or_else(|| anyhow!("workspace member must be a listed package: '{}'", name))
80}80}
81 81
@@ -172,7 +172,7 @@ fn workspace_members_referring_to_us<'a>(meta: &'a Metadata, publishee: &Package · 90 unmodified lines
172 && meta172 && meta
173 .workspace_members173 .workspace_members
174 .iter()174 .iter()
175 .map(|id| id_to_package(&meta, id))175 .map(|id| id_to_package(meta, id))
176 .any(|potential_cycle| potential_cycle.name == dep.name)176 .any(|potential_cycle| potential_cycle.name == dep.name)
177 })177 })
178 .filter_map(|dep| {178 .filter_map(|dep| {
@@ -201,7 +201,7 @@ fn hops_for_dependency_to_link_back_to_us<'a>( · 22 unmodified lines
201 if !seen.insert(name) {201 if !seen.insert(name) {
202 continue;202 continue;
203 }203 }
204 if let Some(package) = workspace_package_by_name(meta, &name) {204 if let Some(package) = workspace_package_by_name(meta, name) {
205 if package.dependencies.iter().any(|dep| dep.name == destination.name) {205 if package.dependencies.iter().any(|dep| dep.name == destination.name) {
206 return Some(level + 1);206 return Some(level + 1);
207 }207 }