check-mirrors: fix minor bugs

* The root progress node should not have an estimated total
* Use `actions/checkout@v3`, not `actions/checkout@v2`
* Do not clean the repository; this removes the local cache, slowing
  down website deploys (since the same runner is used)
* Do not attempt to send a push notification if it fails on a PR

Author: mlugg

.github/workflows/check-mirrors.yml

@@ -7,15 +7,16 @@ jobs:
   check:
     runs-on: [self-hosted, website]
     steps:
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v3
+        with: { clean: false }
       - name: Check Mirrors
         run: |
           cd check-mirrors
           /home/ci/deps/zig-linux-x86_64-0.14.0/zig build run -- ../assets/community-mirrors.ziggy "$GITHUB_STEP_SUMMARY"
   notify-failure:
     runs-on: [self-hosted, website]
     needs: [check]
-    if: ${{ always() && (needs.check.result == 'failure' || needs.check.result == 'timed_out') }}
+    if: ${{ always() && (github.event_name != 'pull_request') && (needs.check.result == 'failure' || needs.check.result == 'timed_out') }}
     steps:
       - name: Send Notification
         env:

check-mirrors/main.zig

@@ -189,10 +189,7 @@ pub fn main() Allocator.Error!u8 {
 
     std.log.info("{d} mirrors; {d} tarballs; {d} signatures", .{ mirrors.len, tarballs.items.len, minisigs.items.len });
 
-    var root_prog_node = std.Progress.start(.{
-        .root_name = "check",
-        .estimated_total_items = tarballs.items.len + minisigs.items.len,
-    });
+    var root_prog_node = std.Progress.start(.{ .root_name = "check" });
     defer root_prog_node.end();
     const tarballs_prog_node = root_prog_node.start("tarballs", tarballs.items.len);
     const minisigs_prog_node = root_prog_node.start("signatures", minisigs.items.len);