.github/FUNDING.yml

@@ -1 +1 @@
-github: [mvdan, lu4p]
+github: [mvdan, lu4p, pagran]

.github/workflows/test.yml

@@ -2,41 +2,74 @@ on:
   push:
     branches:
       - master
+      - ci-test
   pull_request:
     branches:
       - master
 
 # Note that a full "go test" is quite heavy,
 # as it runs many builds under the hood.
 # The default -timeout=10m can be hit by the hosted runners.
+#
+# Also note that we don't use Actions caching for Go on purpose.
+# Caching GOMODCACHE wouldn't help much, as we have few deps.
+# Caching GOCACHE would do more harm than good,
+# as the tests redo most of their work if the garble version changes,
+# and the majority of commits or PRs will do so.
+# Moreover, GitHub saves and restores caches via compressed archives over the
+# network, which means it can easily add one minute of overhead on its own for
+# just a few hundred megabytes worth of files.
 
 name: Test
 jobs:
   test:
     strategy:
       matrix:
-        go-version: [1.17.x]
+        go-version: [1.23.x, 1.24.x]
         os: [ubuntu-latest, macos-latest, windows-latest]
     runs-on: ${{ matrix.os }}
     steps:
-    - name: Install Go
-      uses: actions/setup-go@v2
+    - uses: actions/checkout@v4
+    - uses: actions/setup-go@v5
       with:
         go-version: ${{ matrix.go-version }}
-    - name: Checkout code
-      uses: actions/checkout@v2
-    - name: Test
-      run: |
-        go env
-        go test -timeout=15m ./...
-    - name: Test with -race
-      # macos and windows tend to be a bit slower,
-      # and it's rare that a race in garble would be OS-specific.
-      if: matrix.os == 'ubuntu-latest'
+        cache: false
+    # linux already runs the tests with -race below, plus extra checks,
+    # so skip the regular tests to prevent it from taking the longest.
+    - if: matrix.os != 'ubuntu-latest'
+      run: go test -timeout=15m ./...
+    # macos and windows failures with bincmp can be hard to reproduce locally,
+    # so upload the binaries as artifacts.
+    - uses: actions/upload-artifact@v4
+      if: failure()
+      with:
+        name: bincmp_output
+        path: bincmp_output/
+    # macos and windows tend to be a bit slower,
+    # and it's rare that a race in garble would be OS-specific.
+    - if: matrix.os == 'ubuntu-latest'
+      run: go test -race -timeout=20m ./...
+
+    # Static checks from this point forward. Only run on one Go version and on
+    # linux, since it's the fastest platform, and the tools behave the same.
+    - name: Test third-party project builds
+      if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.24.x'
       run: |
-        go test -race -timeout=20m ./...
+        go install
+        ./scripts/check-third-party.sh
+    - if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.24.x'
+      run: ./scripts/crlf-test.sh
+    - if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.24.x'
+      run: diff <(echo -n) <(gofmt -d .)
+    - if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.24.x'
+      run: go vet ./...
+    - if: matrix.os == 'ubuntu-latest' && matrix.go-version == '1.24.x'
+      uses: dominikh/staticcheck-action@v1
+      with:
+        version: "2025.1"
+        install-go: false
 
-  # We don't care about GOARCH=386 particularly,
+  # We don't care about GOARCH=386 particularly, hence -short,
   # but it helps ensure we support 32-bit hosts and targets well.
   # TODO: use CGO_ENABLED=1 once we figure out how to install gcc-multilib,
   # and once gotooltest forwards the value of CGO_ENABLED to the scripts.
@@ -45,48 +78,9 @@ jobs:
     env:
       GOARCH: 386
     steps:
-    - name: Install Go
-      uses: actions/setup-go@v2
+    - uses: actions/checkout@v4
+    - uses: actions/setup-go@v5
       with:
-        go-version: 1.17.x
-    - name: Checkout code
-      uses: actions/checkout@v2
-    - name: Test
-      run: |
-        go env
-        go test -timeout=15m ./...
-
-  test-gotip:
-    runs-on: ubuntu-latest
-    steps:
-    - name: Install Go
-      env:
-        GO_COMMIT: 3b5eec937018be98549dea7067964018f0e5824c # 2022-01-14
-      run: |
-        cd $HOME
-        mkdir $HOME/gotip
-        cd $HOME/gotip
-
-        wget -O gotip.tar.gz https://go.googlesource.com/go/+archive/${GO_COMMIT}.tar.gz
-        tar -xf gotip.tar.gz
-        echo "devel go1.18-${GO_COMMIT}" >VERSION
-
-        cd src
-        # GOGC=off helps Go build about 20% faster, if we can spare memory.
-        GOGC=off ./make.bash
-        echo "GOROOT=$HOME/gotip" >>$GITHUB_ENV
-        echo "$HOME/gotip/bin" >>$GITHUB_PATH
-    - name: Checkout code
-      uses: actions/checkout@v2
-    - name: Test
-      run: |
-        go env
-        go test -timeout=15m ./...
-
-  code-checks:
-    runs-on: ubuntu-latest
-    steps:
-    - name: Checkout code
-      uses: actions/checkout@v2
-    - name: Test that only LF line endings are used
-      run: ./scripts/crlf-test.sh
+        go-version: 1.24.x
+        cache: false
+    - run: go test -short ./...

.gitignore

@@ -1,2 +1,4 @@
 /garble
-/test
+/test
+/bincmp_output/
+debug

AUTHORS

@@ -8,7 +8,11 @@
 
 Andrew LeFevre <jalefevre@liberty.edu>
 Daniel Martí <mvdan@mvdan.cc>
+Emmanuel Chee-zaram Okeke <ecokeke21@gmail.com>
+NHAS <jordanatararimu@gmail.com>
 Nicholas Jones <me@nicholasjon.es>
+Paul Scheduikat <lu4p@pm.me>
 Zachary Wasserman <zachwass2000@gmail.com>
-lu4p <lu4p@pm.me>
 pagran <pagran@protonmail.com>
+shellhazard <shellhazard@tutanota.com>
+xuannv <xuan11290@gmail.com>