agate

commit 8813196bdbd19c73a2225d4785ceace7d6e0e361
parent aaf3279e95e41020b239158209c99ad2ab49635f
Author: j-k <dev@j-k.io>
Date:   Thu, 14 Oct 2021 19:45:45 +0100

ignore dependabot prs for clippy annotations (#78)


Diffstat:
M
.github/workflows/cargo-audit.yml
 | 
2
+-
M
.github/workflows/test.yml
 | 
16
++++++++++++++--
M
tests/tests.rs
 | 
4
++--

3 files changed, 17 insertions(+), 5 deletions(-)
diff --git a/.github/workflows/cargo-audit.yml b/.github/workflows/cargo-audit.yml
@@ -12,7 +12,7 @@ jobs:
   cargo-audit:
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v1
+      - uses: actions/checkout@v2
       - uses: actions-rs/audit-check@v1
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -2,19 +2,31 @@ name: Tests
 
 on:
   push:
+    branches-ignore:
+      - dependabot/*
   pull_request:
     branches:
       - master
 
 jobs:
-  build:
+  clippy:
     runs-on: ubuntu-latest
     steps:
       - uses: actions/checkout@v2
-      - uses: actions-rs/clippy-check@v1
+      - name: Run clippy action to produce annotations
+        # Don't run on dependabot PRs
+        # https://github.com/actions-rs/clippy-check/issues/2#issuecomment-807852653
+        if: github.event.pull_request.head.repo.full_name == github.repository && github.actor != 'dependabot[bot]'
+        uses: actions-rs/clippy-check@v1
         with:
           token: ${{ secrets.GITHUB_TOKEN }}
           args: --all-features
+      - name: Run clippy manually without annotations
+        if: github.event.pull_request.head.repo.full_name != github.repository || github.actor == 'dependabot[bot]'
+        uses: actions-rs/cargo@v1
+        with:
+          command: clippy
+          args: --all-targets -- -D warnings
   formatting:
     runs-on: ubuntu-latest
     steps:
diff --git a/tests/tests.rs b/tests/tests.rs
@@ -440,7 +440,7 @@ fn directory_traversal_regression() {
     absolute
         .path_segments_mut()
         .unwrap()
-        .push(&env!("CARGO_MANIFEST_DIR")) // separators will be percent-encoded
+        .push(env!("CARGO_MANIFEST_DIR")) // separators will be percent-encoded
         .push("tests")
         .push("data")
         .push("directory_traversal.gmi");
@@ -449,7 +449,7 @@ fn directory_traversal_regression() {
     relative_escape_path.push("testdir");
     relative_escape_path.push("..");
     relative_escape_path.push("..");
-    let mut relative = base.clone();
+    let mut relative = base;
     relative
         .path_segments_mut()
         .unwrap()