about summary refs log tree commit diff
diff options
context:
space:
mode:
authorAlyssa Ross <hi@alyssa.is>2023-10-28 13:20:49 +0200
committerAlyssa Ross <hi@alyssa.is>2023-10-28 13:20:49 +0200
commite3729eb03b391fcc94e39d11c750f96b6cdda4d0 (patch)
tree2fed181459520334fe1c0254b8cf5115d313e2b5
parentbcc2379d5b0debd9cdb2a97d845bf39975540eb1 (diff)
downloadpr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.tar
pr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.tar.gz
pr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.tar.bz2
pr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.tar.lz
pr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.tar.xz
pr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.tar.zst
pr-tracker-e3729eb03b391fcc94e39d11c750f96b6cdda4d0.zip
Fix clippy warnings
-rw-r--r--src/branches.rs3
-rw-r--r--src/main.rs2
-rw-r--r--src/nixpkgs.rs8
3 files changed, 6 insertions, 7 deletions
diff --git a/src/branches.rs b/src/branches.rs
index e3ee262..47bd1b1 100644
--- a/src/branches.rs
+++ b/src/branches.rs
@@ -95,14 +95,13 @@ pub fn branch_hydra_link(branch: &str) -> Option<String> {
         .matches(branch)
         .iter()
         .next()
-        .map(|index| {
+        .and_then(|index| {
             let regex = BRANCH_HYDRA_LINK_PATTERNS.get(index).unwrap();
             BRANCH_HYDRA_LINKS_BY_INDEX
                 .get(index)
                 .map(move |link| regex.replace(branch, *link))
                 .map(move |l| format!("https://hydra.nixos.org/job/{}#tabs-constituents", l))
         })
-        .flatten()
 }
 
 #[cfg(test)]
diff --git a/src/main.rs b/src/main.rs
index 6073b9a..b088e1f 100644
--- a/src/main.rs
+++ b/src/main.rs
@@ -188,7 +188,7 @@ async fn main() {
 
     let mut listeners: Vec<Pin<Box<dyn Future<Output = _>>>> = Vec::new();
 
-    for fd in (3..).into_iter().take(fd_count as usize) {
+    for fd in (3..).take(fd_count as usize) {
         let s = server.clone();
         if handle_error(is_socket_inet(fd), 74, "sd_is_socket_inet") {
             listeners.push(Box::pin(s.listen(unsafe { TcpListener::from_raw_fd(fd) })));
diff --git a/src/nixpkgs.rs b/src/nixpkgs.rs
index 7e622c2..6656543 100644
--- a/src/nixpkgs.rs
+++ b/src/nixpkgs.rs
@@ -56,7 +56,7 @@ impl<'a> Nixpkgs<'a> {
     fn git_command(&self, subcommand: impl AsRef<OsStr>) -> Command {
         let mut command = Command::new("git");
         command.arg("-C");
-        command.arg(&self.path);
+        command.arg(self.path);
         command.arg(subcommand);
         command
     }
@@ -64,7 +64,7 @@ impl<'a> Nixpkgs<'a> {
     async fn git_branch_contains(&self, commit: &str) -> Result<Vec<u8>> {
         let output = self
             .git_command("branch")
-            .args(&["-r", "--format=%(refname)", "--contains"])
+            .args(["-r", "--format=%(refname)", "--contains"])
             .arg(commit)
             .stderr(Stdio::inherit())
             .output()
@@ -79,7 +79,7 @@ impl<'a> Nixpkgs<'a> {
     async fn git_fetch_nixpkgs(&self) -> Result<()> {
         // TODO: add refspecs
         self.git_command("fetch")
-            .arg(&self.remote_name)
+            .arg(self.remote_name)
             .status()
             .await
             .map_err(Error::Io)
@@ -109,7 +109,7 @@ impl<'a> Nixpkgs<'a> {
         };
 
         let mut prefix = PathBuf::from("refs/remotes/");
-        prefix.push(&self.remote_name);
+        prefix.push(self.remote_name);
 
         for branch_name in output
             .split(|byte| *byte == b'\n')