enable deny check

databendlabs · Sep 22, 2024 · fdac659 · fdac659
1 parent 18def51
commit fdac659
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 6 deletions.
diff --git a/.github/actions/check/action.yml b/.github/actions/check/action.yml
@@ -32,10 +32,10 @@ runs:
       run: |
         cargo machete
 
-    # - name: Deny Check
-    #   shell: bash
-    #   run: |
-    #     cargo deny check
+    - name: Deny Check
+      shell: bash
+      run: |
+        cargo deny check
 
     - name: Clippy
       shell: bash

diff --git a/core/src/client.rs b/core/src/client.rs
@@ -660,7 +660,7 @@ impl RouteHintGenerator {
         let uuid = uuid::Uuid::new_v4();
         let current = format!("rh:{}:{:06}", uuid, nonce);
         let mut guard = self.current.lock().unwrap();
-        *guard = current.clone();
+        guard.clone_from(&current);
         current
     }
 }

diff --git a/sql/src/value.rs b/sql/src/value.rs
@@ -943,7 +943,7 @@ pub fn parse_decimal(text: &str, size: DecimalSize) -> Result<NumberValue> {
     }
     let text = &text[start..];
     let point_pos = text.find('.');
-    let e_pos = text.find(|c| c == 'e' || c == 'E');
+    let e_pos = text.find(|c| ['E', 'e'].contains(&c));
     let (i_part, f_part, e_part) = match (point_pos, e_pos) {
         (Some(p1), Some(p2)) => (&text[..p1], &text[(p1 + 1)..p2], Some(&text[(p2 + 1)..])),
         (Some(p), None) => (&text[..p], &text[(p + 1)..], None),