Add comment

.forgejo/workflows/ci.yaml

@@ -14,18 +14,28 @@ jobs:
     steps:
       - uses: actions/checkout@v4
       - name: Run unit tests
-        run: nix develop --command cargo test --lib
+        run: nix build --print-build-logs .#checks.x86_64-linux.default
 
   build:
-    name: Build ${{ matrix.package }}
+    name: Build ${{ matrix.target }}
     runs-on: native
     strategy:
       fail-fast: false
       matrix:
-        package:
+        include:
-          - default
+          - package: x86_64-linux
-          - x86_64-linux-musl
+            target: x86_64-unknown-linux-gnu
-          - aarch64-linux
+          - package: x86_64-linux-musl
+            target: x86_64-unknown-linux-musl
+          - package: aarch64-linux
+            target: aarch64-unknown-linux-gnu
+          # TODO: need macOS runners for these
+          # - package: default
+          #   target: x86_64-apple-darwin
+          #   system: x86_64-darwin
+          # - package: default
+          #   target: aarch64-apple-darwin
+          #   system: aarch64-darwin
     steps:
       - uses: actions/checkout@v4
 
@@ -34,23 +44,14 @@ jobs:
 
       - name: Prepare release artifact
         if: startsWith(forgejo.ref, 'refs/tags/')
-        run: |
+        run: cp result/bin/mkv mkv-${{ matrix.target }}
-          target="${{ matrix.package }}"
-          if [ "$target" = "default" ]; then
-            target="x86_64-unknown-linux-gnu"
-          elif [ "$target" = "x86_64-linux-musl" ]; then
-            target="x86_64-unknown-linux-musl"
-          elif [ "$target" = "aarch64-linux" ]; then
-            target="aarch64-unknown-linux-gnu"
-          fi
-          cp result/bin/mkv "mkv-${target}"
 
       - name: Upload artifact
         if: startsWith(forgejo.ref, 'refs/tags/')
         uses: actions/upload-artifact@v3
         with:
-          name: mkv-${{ matrix.package }}
+          name: mkv-${{ matrix.target }}
-          path: mkv-*
+          path: mkv-${{ matrix.target }}
 
   release:
     name: Create release

flake.nix

@@ -35,11 +35,19 @@
         };
         packages = {
           default = mkv;
+          x86_64-linux = mkv;
         } // pkgs.lib.optionalAttrs pkgs.stdenv.hostPlatform.isLinux {
           x86_64-linux-musl = mkMkv pkgs.pkgsCross.musl64;
           aarch64-linux = mkMkv pkgs.pkgsCross.aarch64-multiplatform;
         };
-        checks.default = mkv.overrideAttrs { doCheck = true; };
+        checks.default = mkv.overrideAttrs {
+          doCheck = true;
+          checkPhase = ''
+            runHook preCheck
+            cargo test --lib
+            runHook postCheck
+          '';
+        };
       }
     );
 }