Merge pull request #13 from Intellection/add_test_and_release_workflows

Add test and release workflows using GitHub Actions
Intellection · Nov 20, 2024 · 81480c1 · 81480c1
2 parents 35d67b2 + d4dbd96
commit 81480c1
Show file tree

Hide file tree

Showing 5 changed files with 110 additions and 2 deletions.
diff --git a/.github/workflows/release.yml b/.github/workflows/release.yml
@@ -0,0 +1,50 @@
+name: release
+
+on:
+  push:
+    tags:
+      - "[0-9].[0-9]+.[0-9]+"
+
+jobs:
+  github-release:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Create Release
+        uses: actions/create-release@v1
+        env:
+          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+        with:
+          tag_name: ${{ github.ref_name }}
+          release_name: Version ${{ github.ref_name }}
+          draft: false
+          prerelease: false
+
+  release-binaries:
+    runs-on: ubuntu-latest
+    needs: github-release
+    strategy:
+      matrix:
+        goos:
+          - linux
+          - darwin
+          - windows
+        goarch:
+          - amd64
+          - arm64
+        exclude:
+          - goarch: arm64
+            goos: windows
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Publish Go Binaries
+        uses: wangyoucao577/[email protected]
+        with:
+          github_token: ${{ secrets.GITHUB_TOKEN }}
+          goos: ${{ matrix.goos }}
+          goarch: ${{ matrix.goarch }}
+          ldflags: -X "main.tagVersion=${{ github.ref_name }}"
+          extra_files: LICENSE README.md
+          md5sum: true
+          sha256sum: true
+          asset_name: resque-exporter-${{ github.ref_name }}-${{ matrix.goos }}-${{ matrix.goarch }}
diff --git a/.github/workflows/test.yml b/.github/workflows/test.yml
@@ -0,0 +1,47 @@
+name: tests
+
+on:
+  pull_request:
+    branches:
+      - '*'
+  push:
+    branches:
+      - master
+
+jobs:
+  linting:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          cache: true
+          cache-dependency-path: go.sum
+          go-version-file: go.mod
+      - name: Run Linter
+        uses: golangci/golangci-lint-action@v6
+        with:
+          version: v1.62.0
+
+  unit-testing:
+    runs-on: ubuntu-latest
+    steps:
+      - name: Checkout
+        uses: actions/checkout@v4
+      - name: Set up Go
+        uses: actions/setup-go@v5
+        with:
+          cache: true
+          cache-dependency-path: go.sum
+          go-version-file: go.mod
+      - name: Build
+        run: go build -v ./...
+      - name: Run Tests
+        run: go test -json ./... > test-results.json
+      - name: Annotate Tests
+        if: always()
+        uses: guyarb/[email protected]
+        with:
+          test-results: test-results.json
diff --git a/.golangci.yml b/.golangci.yml
@@ -0,0 +1,5 @@
+linters:
+  disable-all: true
+  enable:
+    - govet
+    - revive
diff --git a/README.md b/README.md
@@ -1,5 +1,7 @@
 # Resque Exporter
 
+[![tests](https://github.com/Intellection/resque-exporter/actions/workflows/test.yml/badge.svg)](https://github.com/Intellection/resque-exporter/actions/workflows/test.yml)  [![release](https://github.com/Intellection/resque-exporter/actions/workflows/release.yml/badge.svg)](https://github.com/Intellection/resque-exporter/actions/workflows/release.yml)
+
 Prometheus exporter for Resque metrics.
 
 ## Usage

diff --git a/resque_exporter.go b/resque_exporter.go
@@ -350,15 +350,19 @@ func main() {
 	reg.MustRegister(exporter)
 
 	http.Handle(*metricPath, promhttp.HandlerFor(reg, promhttp.HandlerOpts{Registry: reg}))
-	http.HandleFunc("/", func(w http.ResponseWriter, r *http.Request) {
-		w.Write([]byte(`<html>
+	http.HandleFunc("/", func(w http.ResponseWriter, _ *http.Request) {
+		_, err := w.Write([]byte(`<html>
 <head><title>Resque Exporter</title></head>
 <body>
 <h1>Resque Exporter</h1>
 <p><a href='` + *metricPath + `'>Metrics</a></p>
 </body>
 </html>
 `))
+
+		if err != nil {
+			log.Error(err)
+		}
 	})
 
 	log.Infoln("Listening on", *listenAddress)