fix: flip is_ok with is_err for checking early returns

ripytide · Nov 17, 2024 · 6a0421b · 6a0421b
1 parent 2a6feee
commit 6a0421b
Show file tree

Hide file tree

Showing 10 changed files with 11 additions and 11 deletions.
diff --git a/src/backends/apt.rs b/src/backends/apt.rs
@@ -27,7 +27,7 @@ impl Backend for Apt {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/arch.rs b/src/backends/arch.rs
@@ -27,7 +27,7 @@ impl Backend for Arch {
         mut packages: BTreeMap<String, Self::InstallOptions>,
         config: &Config,
     ) -> Result<BTreeMap<String, Self::InstallOptions>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 
@@ -94,7 +94,7 @@ impl Backend for Arch {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/brew.rs b/src/backends/brew.rs
@@ -27,7 +27,7 @@ impl Backend for Brew {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/cargo.rs b/src/backends/cargo.rs
@@ -46,7 +46,7 @@ impl Backend for Cargo {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/dnf.rs b/src/backends/dnf.rs
@@ -31,7 +31,7 @@ impl Backend for Dnf {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/flatpak.rs b/src/backends/flatpak.rs
@@ -30,7 +30,7 @@ impl Backend for Flatpak {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/pipx.rs b/src/backends/pipx.rs
@@ -32,7 +32,7 @@ impl Backend for Pipx {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/rustup.rs b/src/backends/rustup.rs
@@ -35,7 +35,7 @@ impl Backend for Rustup {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/winget.rs b/src/backends/winget.rs
@@ -29,7 +29,7 @@ impl Backend for WinGet {
     }
 
     fn query_installed_packages(config: &Config) -> Result<BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }
 

diff --git a/src/backends/xbps.rs b/src/backends/xbps.rs
@@ -31,7 +31,7 @@ impl Backend for Xbps {
     fn query_installed_packages(
         config: &Config,
     ) -> Result<std::collections::BTreeMap<String, Self::QueryInfo>> {
-        if Self::version(config).is_ok() {
+        if Self::version(config).is_err() {
             return Ok(BTreeMap::new());
         }