Merge pull request #36 from kymppi/update-workflow

chore(workflow): update workflow scrips

.github/workflows/dependencies.yml

@@ -0,0 +1,24 @@
+name: Dependency check
+on:
+  pull_request:
+    paths:
+      - '**/Cargo.lock'
+      - '**/Cargo.toml'
+      - '**/deny.toml'
+jobs:
+  cargo-deny:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        checks:
+          - advisories
+          - bans licenses sources
+
+    # Prevent sudden announcement of a new advisory from failing ci:
+    continue-on-error: ${{ matrix.checks == 'advisories' }}
+
+    steps:
+      - uses: actions/checkout@v4
+      - uses: EmbarkStudios/cargo-deny-action@v1
+        with:
+          command: check ${{ matrix.checks }}

.github/workflows/linting.yml

@@ -1,40 +1,27 @@
 name: Linting
 on: [push, pull_request]
 
+env:
+  RUSTFLAGS: '-Dwarnings'
+
 jobs:
-  cargo-deny:
-    name: Cargo dependency linting
-    runs-on: ubuntu-latest
-    steps:
-      - uses: actions/checkout@v2
-      - uses: EmbarkStudios/cargo-deny-action@v1
   clippy:
     name: Clippy linting
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions-rs/toolchain@v1
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@nightly
         with:
-          toolchain: nightly
           components: clippy
-          override: true
-      - uses: actions-rs/clippy-check@v1
-        with:
-          token: ${{ secrets.GITHUB_TOKEN }}
-          args: --all-features
-          name: Clippy Output
+      - name: Run clippy
+        run: cargo clippy --all-targets --all-features
   fmt:
     name: Check formatting
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions-rs/toolchain@v1
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@nightly
         with:
-          profile: minimal
-          toolchain: nightly
           components: rustfmt
-          override: true
-      - uses: actions-rs/cargo@v1
-        with:
-          command: fmt
-          args: --all -- --check
+      - name: Check formatting
+        run: cargo fmt --all -- --check

.github/workflows/testing.yml

@@ -12,12 +12,9 @@ jobs:
           - stable
           - 1.63.0
     steps:
-      - uses: actions/checkout@v2
-      - uses: actions-rs/toolchain@v1
+      - uses: actions/checkout@v4
+      - uses: dtolnay/rust-toolchain@master
         with:
-          profile: minimal
           toolchain: ${{ matrix.rust }}
-          override: true
-      - uses: actions-rs/cargo@v1
-        with:
-          command: test
+      - name: Run tests
+        run: cargo test

src/layouts.rs

@@ -167,7 +167,7 @@ impl Layouts {
             .await?;
 
         match result {
-            Response::Success(data) => Ok(()),
+            Response::Success(_data) => Ok(()),
             Response::Error(err) => match err.status_code {
                 401 => Err(NovuError::UnauthorizedError("/layouts".to_string())),
                 code => todo!("{}", code),