Merge pull request #321 from mrueg/github-actions

Move to Github Actions / Golangci-Lint

Author: serathius

.github/workflows/tests.yaml

@@ -0,0 +1,16 @@
+name: Tests
+on: [push, pull_request]
+jobs:
+  test:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: actions/checkout@v2
+    - uses: actions/setup-go@v2
+      with:
+        go-version: "1.17.8"
+    - run: make fmt
+    - run: make race
+    - run: make test
+    # Enable when fixed (same as make lint)
+    # - name: golangci-lint
+    #   uses: golangci/golangci-lint-action@v2

.travis.yml

@@ -10,17 +10,8 @@ race:
 fmt:
 	!(gofmt -l -s -d $(shell find . -name \*.go) | grep '[a-z]')
 
-# go get honnef.co/go/tools/simple
-gosimple:
-	gosimple ./...
-
-# go get honnef.co/go/tools/unused
-unused:
-	unused ./...
-
-# go get github.com/kisielk/errcheck
-errcheck:
-	@errcheck -ignorepkg=bytes -ignore=os:Remove go.etcd.io/bbolt
+lint:
+	golangci-lint run ./...
 
 test:
 	TEST_FREELIST_TYPE=hashmap go test -timeout 20m -v -coverprofile cover.out -covermode atomic
@@ -33,4 +24,4 @@ test:
 	# Note: gets "program not an importable package" in out of path builds
 	@TEST_FREELIST_TYPE=array go test -v ./cmd/bbolt
 
-.PHONY: race fmt errcheck test gosimple unused
+.PHONY: race fmt test lint

Makefile

@@ -30,15 +30,15 @@ const pageHeaderSize = 16
 
 // meta represents a simplified version of a database meta page for testing.
 type meta struct {
-	magic    uint32
+	_        uint32
 	version  uint32
 	_        uint32
 	_        uint32
 	_        [16]byte
 	_        uint64
 	pgid     uint64
 	_        uint64
-	checksum uint64
+	_        uint64
 }
 
 // Ensure that a database can be opened without error.