add codecov to ci workflow (#118)

* add codecov to ci workflow

* clippy

Author: mfreeborn

.github/workflows/ci.yml

@@ -50,6 +50,18 @@ jobs:
       - uses: actions/checkout@v3
       - uses: dtolnay/rust-toolchain@stable
       - run: cargo test --features=plotly_ndarray,kaleido
+  
+  code-coverage:
+    name: Code Coverage
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v3
+      - uses: dtolnay/rust-toolchain@stable
+        with:
+          components: llvm-tools-preview
+      - uses: taiki-e/install-action@cargo-llvm-cov
+      - run: cargo llvm-cov --workspace --lcov --output-path lcov.info
+      - uses: codecov/codecov-action@v3
 
   build_examples:
     name: Build Examples

plotly/src/plot.rs

@@ -445,7 +445,7 @@ impl Plot {
     fn show_with_default_app(temp_path: &str) {
         use std::process::Command;
         Command::new("xdg-open")
-            .args(&[temp_path])
+            .args([temp_path])
             .output()
             .expect(DEFAULT_HTML_APP_NOT_FOUND);
     }

plotly_derive/src/field_setter.rs

@@ -155,7 +155,7 @@ fn _type_str_parts(field_ty: &syn::Type) -> (Vec<String>, Vec<syn::Type>) {
     types.push(ty.clone());
     loop {
         match ty {
-            syn::Type::Path(ref type_path) if type_path.qself == None => {
+            syn::Type::Path(ref type_path) if type_path.qself.is_none() => {
                 if let Some(segment) = type_path.path.segments.last() {
                     parts.push(segment.ident.to_string());
                     match &segment.arguments {

plotly_kaleido/build.rs

@@ -36,7 +36,7 @@ const KALEIDO_BIN: &str = "kaleido.exe";
 const KALEIDO_BIN: &str = "kaleido";
 
 fn extract_zip(p: &Path, zip_file: &Path) -> Result<()> {
-    let file = fs::File::open(&zip_file).unwrap();
+    let file = fs::File::open(zip_file).unwrap();
     let mut archive = zip::ZipArchive::new(file).unwrap();
 
     for i in 0..archive.len() {
@@ -68,7 +68,7 @@ fn extract_zip(p: &Path, zip_file: &Path) -> Result<()> {
             );
             if let Some(p) = outpath.parent() {
                 if !p.exists() {
-                    fs::create_dir_all(&p).unwrap();
+                    fs::create_dir_all(p).unwrap();
                 }
             }
             let mut outfile = fs::File::create(&outpath).unwrap();
@@ -104,13 +104,13 @@ fn main() -> Result<()> {
     let kaleido_zip_file = p.join("kaleido.zip");
 
     let mut cmd = Command::new("cargo")
-        .args(&["install", "ruget"])
+        .args(["install", "ruget"])
         .spawn()
         .unwrap();
     cmd.wait()?;
 
     let mut cmd = Command::new("ruget")
-        .args(&[
+        .args([
             KALEIDO_URL,
             "-o",
             kaleido_zip_file.as_path().to_str().unwrap(),

plotly_kaleido/src/lib.rs

@@ -139,7 +139,7 @@ impl Kaleido {
 
         let process = Command::new(p.as_str())
             .current_dir(self.cmd_path.parent().unwrap())
-            .args(&[
+            .args([
                 "plotly",
                 "--disable-gpu",
                 "--allow-file-access-from-files",