remove sentry test for macOS, remove windows check

.github/workflows/test.yml

@@ -381,7 +381,7 @@ jobs:
       fail-fast: false
       matrix:
         sentry: ["true", "false"]
-    name: Build -Dsentry=${{ matrix.sentry }} on Linux
+    name: Build -Dsentry=${{ matrix.sentry }}
     runs-on: namespace-profile-ghostty-sm
     needs: test
     env:
@@ -411,31 +411,6 @@ jobs:
         run: |
           nix develop -c zig build -Dsentry=${{ matrix.sentry }}
 
-  test-sentry-macos:
-    strategy:
-      fail-fast: false
-      matrix:
-        sentry: ["true", "false"]
-    name: Build -Dsentry=${{ matrix.sentry }} on macOS
-    runs-on: namespace-profile-ghostty-macos
-    needs: test
-    steps:
-      - name: Checkout code
-        uses: actions/checkout@v4
-
-      # Install Nix and use that to run our tests so our environment matches exactly.
-      - uses: cachix/install-nix-action@v30
-        with:
-          nix_path: nixpkgs=channel:nixos-unstable
-      - uses: cachix/cachix-action@v15
-        with:
-          name: ghostty
-          authToken: "${{ secrets.CACHIX_AUTH_TOKEN }}"
-
-      - name: Test Sentry Build
-        run: |
-          nix develop -c zig build -Dsentry=${{ matrix.sentry }}
-
   test-macos:
     runs-on: namespace-profile-ghostty-macos
     needs: test

build.zig

@@ -159,6 +159,9 @@ pub fn build(b: *std.Build) !void {
     ) orelse sentry: {
         switch (target.result.os.tag) {
             .macos, .ios => break :sentry true,
+
+            // Note its false for linux because the crash reports on Linux
+            // don't have much useful information.
             else => break :sentry false,
         }
     };
@@ -1256,9 +1259,7 @@ fn addDeps(
     }
 
     // Sentry
-    if (config.sentry) sentry: {
+    if (config.sentry) {
-        if (target.result.os.tag == .windows) break :sentry;
-
         const sentry_dep = b.dependency("sentry", .{
             .target = target,
             .optimize = optimize,