Merge pull request #7 from fredbi/ci-linting

updated CI

Author: fredbi

.golangci.yml

@@ -0,0 +1,41 @@
+linters-settings:
+  govet:
+    check-shadowing: true
+  golint:
+    min-confidence: 0
+  gocyclo:
+    min-complexity: 30
+  maligned:
+    suggest-new: true
+  dupl:
+    threshold: 100
+  goconst:
+    min-len: 2
+    min-occurrences: 4
+linters:
+  enable-all: true
+  disable:
+    - maligned
+    - lll
+    - gochecknoglobals
+    - godox
+    - gocognit
+    - whitespace
+    - wsl
+    - funlen
+    - gochecknoglobals
+    - gochecknoinits
+    - scopelint
+    - wrapcheck
+    - exhaustivestruct
+    - exhaustive
+    - nlreturn
+    - testpackage
+    - gci
+    - gofumpt
+    - goerr113
+    - gomnd
+    - tparallel
+    - nestif
+    - godot
+    - errorlint

.travis.yml

@@ -1,10 +1,19 @@
 after_success:
 - bash <(curl -s https://codecov.io/bash)
 go:
-- 1.11.x
-- 1.12.x
+- 1.14.x
+- 1.x
 install:
-- GO111MODULE=off go get -u gotest.tools/gotestsum
+- go get gotest.tools/gotestsum
+jobs:
+  include:
+  # include linting job, but only for latest go version and amd64 arch
+  - go: 1.x
+    arch: amd64
+    install:
+      go get github.com/golangci/golangci-lint/cmd/golangci-lint
+    script:
+    - golangci-lint run --new-from-rev master
 env:
 - GO111MODULE=on
 language: go

reference_test.go

@@ -47,6 +47,7 @@ func TestIsRoot(t *testing.T) {
 	}))
 }
 
+// nolint: dupl
 func TestFull(t *testing.T) {
 
 	in := "http://host/path/a/b/c#/f/a/b"
@@ -81,6 +82,7 @@ func TestFull(t *testing.T) {
 	}
 }
 
+// nolint: dupl
 func TestFullURL(t *testing.T) {
 
 	in := "http://host/path/a/b/c"
@@ -156,6 +158,7 @@ func TestFragmentOnly(t *testing.T) {
 	assert.Equal(t, r3.String(), in[1:])
 }
 
+// nolint: dupl
 func TestURLPathOnly(t *testing.T) {
 
 	in := "/documents/document.json"
@@ -190,6 +193,7 @@ func TestURLPathOnly(t *testing.T) {
 	}
 }
 
+// nolint: dupl
 func TestURLRelativePathOnly(t *testing.T) {
 
 	in := "document.json"
@@ -393,13 +397,12 @@ func TestReferenceResolution(t *testing.T) {
 		"g#s/../x", "http://a/b/c/g#s/../x",
 
 		"http:g", "http:g", // for strict parsers
-		//"http:g", "http://a/b/c/g", // for backward compatibility
+		// "http:g", "http://a/b/c/g", // for backward compatibility
 
 	}
 	for i := 0; i < len(checks); i += 2 {
 		child := checks[i]
 		expected := checks[i+1]
-		// fmt.Printf("%d:   %v  ->  %v\n", i/2, child, expected)
 
 		childRef, e := New(child)
 		if e != nil {