From 7cae2c5e22523a3b3355b0baccce63bc93a82c76 Mon Sep 17 00:00:00 2001
From: Paul Campbell <campbjpa@amazon.com>
Date: Sat, 21 Sep 2024 22:07:04 +0100
Subject: [PATCH] build: add test and build checks to workflows

---
 .forgejo/workflows/push-next.yml | 39 +++++++++++++++++++++++++++++++-
 src/scanner.rs                   |  2 --
 2 files changed, 38 insertions(+), 3 deletions(-)

diff --git a/.forgejo/workflows/push-next.yml b/.forgejo/workflows/push-next.yml
index cc8498c..195deef 100644
--- a/.forgejo/workflows/push-next.yml
+++ b/.forgejo/workflows/push-next.yml
@@ -4,16 +4,53 @@ on:
   push:
     branches:
       - next
+  pull_request:
+    branches:
+      - main
   workflow_dispatch:
 
+env:
+  CARGO_TERM_COLOR: always
+
 jobs:
 
-  test:
+  checks:
     runs-on: docker
 
+    strategy:
+      matrix:
+        toolchain:
+          - name: stable
+          - name: nightly
+
     steps:
       - name: Checkout
         uses: actions/checkout@v4
 
       - name: Check TODOs
         uses: https://git.kemitix.net/kemitix/forgejo-todo-checker@v1
+
+      - name: Machete
+        uses: https://git.kemitix.net/kemitix/rust@v2.1.0
+        with:
+          args: ${{ matrix.toolchain.name }} cargo machete
+
+      - name: Format
+        uses: https://git.kemitix.net/kemitix/rust@v2.1.0
+        with:
+          args: ${{ matrix.toolchain.name }} cargo fmt --all -- --check
+
+      - name: Clippy
+        uses: https://git.kemitix.net/kemitix/rust@v2.1.0
+        with:
+          args: ${{ matrix.toolchain.name }} cargo hack --feature-powerset clippy
+
+      - name: Build
+        uses: https://git.kemitix.net/kemitix/rust@v2.1.0
+        with:
+          args: ${{ matrix.toolchain.name }} cargo hack --feature-powerset build
+
+      - name: Test
+        uses: https://git.kemitix.net/kemitix/rust@v2.1.0
+        with:
+          args: ${{ matrix.toolchain.name }} cargo hack --feature-powerset test
diff --git a/src/scanner.rs b/src/scanner.rs
index b252ecb..085f53b 100644
--- a/src/scanner.rs
+++ b/src/scanner.rs
@@ -72,12 +72,10 @@ impl FileScanner for DefaultFileScanner {
             .for_each(|marker| match marker {
                 Marker::Invalid(_) => {
                     errors += 1;
-                    // TODO: (#6) Better error message
                     printer.println(marker.to_string());
                 }
                 Marker::Closed(_, _) => {
                     errors += 1;
-                    // TODO: (#6) Better error message
                     printer.println(marker.to_string());
                 }
                 _ => {}