chore: update ci and linter (#9)

Author: nrwiersma

.github/workflows/test.yml

@@ -0,0 +1,49 @@
+on:
+  push:
+    branches:
+      - master
+  pull_request:
+
+name: run tests
+jobs:
+  test:
+
+    strategy:
+      matrix:
+        go-version: [ 1.15, 1.16 ]
+    runs-on: ubuntu-latest
+    env:
+      GOLANGCI_LINT_VERSION: v1.41.1
+
+    steps:
+      - name: Install Go
+        if: success()
+        uses: actions/setup-go@v2
+        with:
+          go-version: ${{ matrix.go-version }}
+
+      - name: Checkout code
+        uses: actions/checkout@v2
+
+      - name: Cache Go modules
+        uses: actions/cache@v2
+        with:
+          path: ~/go/pkg/mod
+          key: ${{ runner.os }}-go-${{ hashFiles('**/go.sum') }}
+          restore-keys: |
+            ${{ runner.os }}-go-
+      - name: Run linter
+        uses: golangci/golangci-lint-action@v2
+        with:
+          version: ${{ env.GOLANGCI_LINT_VERSION }}
+
+      - name: Run tests
+        run: go test -covermode=count -coverprofile=coverage.out ./...
+
+      - name: Convert coverage.out to coverage.lcov
+        uses: jandelgado/gcov2lcov-action@v1.0.6
+      - name: Coveralls
+        uses: coverallsapp/github-action@v1.1.2
+        with:
+          github-token: ${{ secrets.GITHUB_TOKEN }}
+          path-to-lcov: coverage.lcov

.golangci.yml

@@ -0,0 +1,36 @@
+run:
+  tests: false
+  timeout: 5m
+
+linters-settings:
+  cyclop:
+    max-complexity: 12
+    skip-tests: true
+  funlen:
+    lines: 80
+  gofumpt:
+    extra-rules: true
+  gosec:
+    excludes:
+      - G103
+
+linters:
+  enable-all: true
+  disable:
+    - interfacer # deprecated
+    - scopelint # deprecated
+    - maligned # deprecated
+    - golint # deprecated
+    - exhaustive
+    - exhaustivestruct
+    - gci
+    - gochecknoglobals
+    - goerr113
+    - gomnd
+    - nlreturn
+    - maligned
+    - wrapcheck
+    - wsl
+
+issues:
+  exclude-use-default: false

.travis.yml

@@ -1,6 +1,6 @@
 MIT License
 
-Copyright (c) 2019 Nicholas Wiersma
+Copyright (c) 2021 Nicholas Wiersma
 
 Permission is hereby granted, free of charge, to any person obtaining a copy
 of this software and associated documentation files (the "Software"), to deal

LICENCE

@@ -1 +1,6 @@
 include github.com/hamba/make/golang
+
+# Run benchmarks
+bench:
+	@go test -bench . ./...
+.PHONY: bench

Makefile

@@ -8,60 +8,88 @@ import (
 	"github.com/hamba/timex/mono"
 )
 
-func BenchmarkMonoNow(b *testing.B) {
+func BenchmarkTimexMonoNow(b *testing.B) {
 	b.ReportAllocs()
-	for i := 0; i < b.N; i++ {
-		_ = mono.Now()
-	}
+	b.ResetTimer()
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = mono.Now()
+		}
+	})
 }
 
-func BenchmarkMonoSince(b *testing.B) {
+func BenchmarkTimexMonoSince(b *testing.B) {
 	start := mono.Now()
 
 	b.ReportAllocs()
 	b.ResetTimer()
-	for i := 0; i < b.N; i++ {
-		_ = mono.Since(start)
-	}
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = mono.Since(start)
+		}
+	})
 }
 
 func BenchmarkTimexNow(b *testing.B) {
 	b.ReportAllocs()
-	for i := 0; i < b.N; i++ {
-		_ = timex.Now()
-	}
+	b.ResetTimer()
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = timex.Now()
+		}
+	})
 }
 
 func BenchmarkTimexSince(b *testing.B) {
 	start := timex.Now()
 
 	b.ReportAllocs()
 	b.ResetTimer()
-	for i := 0; i < b.N; i++ {
-		_ = timex.Since(start)
-	}
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = timex.Since(start)
+		}
+	})
 }
 
 func BenchmarkTimexUnix(b *testing.B) {
 	b.ReportAllocs()
-	for i := 0; i < b.N; i++ {
-		_ = timex.Unix()
-	}
+	b.ResetTimer()
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = timex.Unix()
+		}
+	})
 }
 
-func BenchmarkWallTimeNow(b *testing.B) {
+func BenchmarkTimeNow(b *testing.B) {
 	b.ReportAllocs()
-	for i := 0; i < b.N; i++ {
-		_ = time.Now().UnixNano()
-	}
+	b.ResetTimer()
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = time.Now().UnixNano()
+		}
+	})
 }
 
-func BenchmarkWallTimeSince(b *testing.B) {
+func BenchmarkTimeSince(b *testing.B) {
 	start := time.Now()
 
 	b.ReportAllocs()
 	b.ResetTimer()
-	for i := 0; i < b.N; i++ {
-		_ = time.Since(start)
-	}
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = time.Since(start)
+		}
+	})
+}
+
+func BenchmarkTimeUnix(b *testing.B) {
+	b.ReportAllocs()
+	b.ResetTimer()
+	b.RunParallel(func(pb *testing.PB) {
+		for pb.Next() {
+			_ = time.Now().Unix()
+		}
+	})
 }

bench_test.go

@@ -3,7 +3,8 @@ package mono
 
 import (
 	"time"
-	_ "unsafe" // Required in order to import nanotime
+	// Required in order to import nanotime.
+	_ "unsafe"
 )
 
 //go:linkname nanotime runtime.nanotime

mono/time.go

@@ -3,7 +3,8 @@ package timex
 
 import (
 	"time"
-	_ "unsafe" // Required in order to import walltime
+	// Required in order to import walltime.
+	_ "unsafe"
 )
 
 //go:linkname walltime runtime.walltime