Merge pull request #203 from nicklan/cut-0.6.2

CHANGELOG.md

@@ -1,3 +1,12 @@
+0.6.2
+=====
+Features:
+* Allow setting the path to the kubectl binary that click uses via `set kubectl_binary <path>`
+* Print the full path the kubectl binary in `env`
+
+Bug Fixes:
+* Don't panic if an "azure-auth-provider" doesn't include `expires-by`
+
 0.6.1
 =====
 Bugfix release:

Cargo.toml

@@ -1,6 +1,6 @@
 [package]
 name = "click"
-version = "0.6.1"
+version = "0.6.2"
 description = "A command-line REPL for Kubernetes that integrates into existing cli workflows"
 authors = ["Nick Lanham <nick@afternight.org>"]
 homepage = "https://github.com/databricks/click/wiki"

src/completer.rs

@@ -83,12 +83,7 @@ impl ClickHelper {
 
     #[allow(clippy::borrowed_box)]
     fn get_exact_command(&self, line: &str) -> Option<&Box<dyn Cmd>> {
-        for cmd in self.commands.iter() {
-            if cmd.is(line) {
-                return Some(cmd);
-            }
-        }
-        None
+        self.commands.iter().find(|&cmd| cmd.is(line))
     }
 
     /// complete a line that starts with a full command. This should only be called when we know

src/table.rs

@@ -379,7 +379,7 @@ pub fn raw_quantity(quantity: &Quantity) -> f64 {
 
     if suffix.len() > 1 && (suffix.starts_with('e') || suffix.starts_with('E')) {
         // our suffix has more than one char and starts with e/E, so it should be a decimal exponent
-        match (&suffix[1..]).parse::<u32>() {
+        match (suffix[1..]).parse::<u32>() {
             Ok(exp) => {
                 let famt = (amt * base10.pow(exp)) as f64;
                 if has_neg {