Merge branch 'main' into test-ci

Author: vladopajic

.github/workflows/action-docker-test.yml

@@ -19,10 +19,17 @@ jobs:
         username: ${{ github.repository_owner }}
         password: ${{ secrets.GITHUB_TOKEN }}
 
+    - name: Set up Docker Buildx
+      uses: docker/[email protected]
+      with:
+        use: true
+        platforms: linux/arm64,linux/amd64
+
     - name: build and push
       uses: docker/build-push-action@v6
       with:
         push: true
+        platforms: linux/arm64,linux/amd64
         build-args: |
           VERSION=dev
         tags: |
@@ -39,6 +46,9 @@ jobs:
     name: test
     runs-on: ubuntu-latest
     needs: build-dev-image
+    strategy:
+      matrix:
+        os: [ ubuntu-latest, ubuntu-24.04-arm ]
 
     steps:
       - name: checkout

.github/workflows/release.yml

@@ -40,12 +40,19 @@ jobs:
           username: ${{ github.repository_owner }}
           password: ${{ secrets.GITHUB_TOKEN }}
 
+      - name: Set up Docker Buildx
+        uses: docker/[email protected]
+        with:
+          use: true
+          platforms: linux/arm64,linux/amd64
+
       - name: build and push
         uses: docker/build-push-action@v6
         with:
           push: true
           build-args: |
             VERSION=${{ env.RELEASE_VERSION }}
+          platforms: linux/arm64,linux/amd64
           tags: |
             ghcr.io/vladopajic/go-test-coverage:${{ env.RELEASE_VERSION }}
             ghcr.io/vladopajic/go-test-coverage:${{ steps.majorver.outputs.major-tag }}

Dockerfile

@@ -1,5 +1,6 @@
 # GO_VERSION: automatically update to most recent via dependabot
-FROM golang:1.25.3 AS builder
+FROM --platform=$BUILDPLATFORM golang:1.25.3 AS builder
+
 WORKDIR /workspace
 
 COPY go.mod go.mod
@@ -10,10 +11,13 @@ RUN go mod download all
 COPY ./ ./
 
 ARG VERSION
-RUN CGO_ENABLED=0 GOOS=linux GOARCH=amd64 \
+ARG TARGETOS 
+ARG TARGETARCH
+
+RUN CGO_ENABLED=0 GOOS=$TARGETOS GOARCH=$TARGETARCH \
     go build -o go-test-coverage .
 
-FROM gcr.io/distroless/base:latest
+FROM --platform=$BUILDPLATFORM gcr.io/distroless/base:latest
 WORKDIR /
 COPY --from=builder /workspace/go-test-coverage .
 COPY --from=builder /usr/local/go/bin/go /usr/local/go/bin/go

action.yml

@@ -139,7 +139,7 @@ outputs:
 runs:
   using: docker
   # TOOL_VERSION: when changing version update version in other places
-  image: docker://ghcr.io/vladopajic/go-test-coverage:v2.17.1
+  image: docker://ghcr.io/vladopajic/go-test-coverage:v2.18.0
   args:
     - --config "${{ inputs.config }}"
     - --profile "${{ inputs.profile }}"

action/source/action.yml

@@ -130,7 +130,7 @@ inputs:
     description: Version of go-test-coverage source to run
     required: false
     # TOOL_VERSION: when changing version update version in other places
-    default: v2.17.1
+    default: v2.18.0
     type: string
 
 outputs:

docs/github_action.md

@@ -176,6 +176,10 @@ The `go-test-coverage` project provides two types of GitHub Actions:
     uses: vladopajic/go-test-coverage@v2
     with: ...
   ```
+  Pros/Cons:
+    - Generally faster in practice because everything is packaged and prebuilt.
+    - Does not require any additional dependencies on the CI runner.
+    - However, **it introduces a potential supply-chain risk**: using a prebuilt container image means you must trust that the image is safe and has not been tampered with. This repository has no intention of introducing malicious code, but projects with stricter security requirements may still prefer to avoid running opaque binaries.
 
 - **Source-based Action (optional/experimental)**
 
@@ -188,9 +192,17 @@ The `go-test-coverage` project provides two types of GitHub Actions:
     uses: vladopajic/go-test-coverage/action/source@v2
     with: ...
   ```
-   Note: this action requires `go` to be installed.
+   Pros/Cons:
+   - Requires `go` to be installed on the CI runner.
+   - Slightly slower, as dependencies need to be fetched and the binary built during the workflow.
+   - The main **advantage is transparency and auditability**: the action runs exactly the code in this repository, which can be reviewed and pinned to a specific version. This can be preferable for environments with strong security or compliance requirements.
+
+Both actions have the same inputs, so they can be used interchangeably. 
+
+
+
+
 
-Both actions have the same inputs, so they can be used interchangeably.
 
 
 

main.go

@@ -9,7 +9,7 @@ import (
 )
 
 const (
-	Version = "v2.17.1" // TOOL_VERSION: when changing version update version in other places
+	Version = "v2.18.0" // TOOL_VERSION: when changing version update version in other places
 	Name    = "go-test-coverage"
 )
 

pkg/testcoverage/report.go

@@ -77,11 +77,12 @@ func reportUncoveredLines(w io.Writer, result AnalyzeResult) {
 	defer tabber.Flush()
 
 	fmt.Fprintf(tabber, "\nFiles with uncovered lines:")
-	fmt.Fprintf(tabber, "\n  file:\tuncovered lines:")
+	fmt.Fprintf(tabber, "\n  file:\tcoverage:\tuncovered lines:")
 
 	for _, stats := range result.FilesWithUncoveredLines {
 		if len(stats.UncoveredLines) > 0 {
 			fmt.Fprintf(tabber, "\n  %s\t", stats.Name)
+			fmt.Fprintf(tabber, "%v%%\t", stats.CoveredPercentageF())
 			compressUncoveredLines(tabber, stats.UncoveredLines)
 		}
 	}

pkg/testcoverage/report_test.go

@@ -413,11 +413,15 @@ func Test_ReportUncoveredLines(t *testing.T) {
 		FilesWithUncoveredLines: []coverage.Stats{
 			{Name: "a.go", UncoveredLines: []int{1, 2, 3}},
 			{Name: "b.go", UncoveredLines: []int{3, 5, 7}},
+			{Name: "c.go", UncoveredLines: []int{1, 4, 10}, Covered: 20, Total: 100},
+			{Name: "d.go", UncoveredLines: []int{7, 8, 9}, Covered: 18, Total: 88},
 		},
 	})
 	assertHasUncoveredLinesInfo(t, buf.String(), []string{
-		"a.go\t\t1-3\n",
-		"b.go\t\t3 5 7\n",
+		"a.go\t\t0%\t\t1-3\n",
+		"b.go\t\t0%\t\t3 5 7\n",
+		"c.go\t\t20%\t\t1 4 10\n",
+		"d.go\t\t20.5%\t\t7-9\n",
 	})
 
 	// when result passes, there should be no output