Ensure no dbg! makes it to prod

Also nuke a couple of dbg!() calls in tests that aren't required.
enquo · Oct 7, 2022 · 9221b9c · 9221b9c
1 parent 08acb85
commit 9221b9c
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 4 deletions.
diff --git a/.github/workflows/qc_ruby.yml b/.github/workflows/qc_ruby.yml
@@ -87,4 +87,4 @@ jobs:
         # The `-A unused-variables` here is because in-function use of rbself in methods!
         # doesn't get picked up, and Clippy chucks a wobbly
         run: |
-          cargo clippy -- -D warnings -A unused-variables
+          cargo clippy -- -D clippy::dbg_macro -D warnings -A unused-variables
diff --git a/.github/workflows/qc_rust.yml b/.github/workflows/qc_rust.yml
@@ -68,4 +68,4 @@ jobs:
       - name: Consult Clippy
         working-directory: rust
         run: |
-          cargo clippy -- -D warnings
+          cargo clippy -- -D clippy::dbg_macro -D warnings
diff --git a/rust/src/datatype/date_v1.rs b/rust/src/datatype/date_v1.rs
@@ -181,7 +181,6 @@ mod tests {
 
         let mut s: Vec<u8> = vec![];
         ciborium::ser::into_writer(&serde_value, &mut s).unwrap();
-        dbg!(&s);
         assert!(s.len() < 300, "s.len() == {}", s.len());
     }
 

diff --git a/rust/src/datatype/i64v1.rs b/rust/src/datatype/i64v1.rs
@@ -141,7 +141,6 @@ mod tests {
 
         let mut s: Vec<u8> = vec![];
         ciborium::ser::into_writer(&serde_value, &mut s).unwrap();
-        dbg!(&s);
         assert!(s.len() < 600, "s.len() == {}", s.len());
     }