Migrate to GitHub Actions

* Use currently supported Go versions only
* Added "build" to verify it builds correctly
* `golint` and `megacheck` are deprecated. Dropped in favour of `go tool
  vet` and `staticcheck`.
* Minor changes to make sure some checks pass

Author: rail

.github/workflows/ci.yaml

@@ -0,0 +1,58 @@
+name: Go
+
+on:
+  push:
+    branches: [ master ]
+  pull_request:
+    branches: [ master ]
+
+jobs:
+
+  build-and-test:
+    runs-on: ubuntu-latest
+    strategy:
+      matrix:
+        go:
+          - "1.16"
+          - "1.17"
+    steps:
+    - uses: actions/checkout@v2
+
+    - name: Set up Go ${{ matrix.go }}
+      uses: actions/setup-go@v2
+      with:
+        go-version: ${{ matrix.go }}
+
+    - name: Install dependencies
+      run: |
+        go install github.com/cockroachdb/crlfmt@latest
+        go install github.com/kisielk/errcheck@latest
+        go install github.com/mdempsky/unconvert@latest
+        go install honnef.co/go/tools/cmd/staticcheck@latest
+        go install golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow@latest
+
+
+    - name: Build
+      run: go build -v ./...
+
+    - name: Test
+      run: go test -v ./...
+    - name: Race
+      run:  go test -p 1 -v ./...
+    - name: gofmt
+      run: gofmt -s -d -l .
+    - name: vet
+      run: |
+        ! go vet -vettool=$(which shadow) ./... 2>&1 | \
+          grep -vF 'declaration of "err" shadows declaration at' | \
+          grep -vF "# github.com/cockroachdb/cockroach-go/v2/testserver"
+    - name: License checks
+      run: |
+        ! git grep -lE '^// Author' -- '*.go'
+        ! git grep -LE '^// Copyright' -- '*.go'
+    - name: errchk
+      run: errcheck -ignore "Close|Init|AutoMigrate"  ./...
+    - name: unconvert
+      run: unconvert ./...
+    - name: staticcheck
+      run: staticcheck ./...

.travis.yml

@@ -1,3 +1,17 @@
+// Copyright 2016 The Cockroach Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+// implied. See the License for the specific language governing
+// permissions and limitations under the License.
+
 package crdbsqlx
 
 import (

crdb/crdbsqlx/sqlx.go

@@ -1,3 +1,17 @@
+// Copyright 2016 The Cockroach Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+// implied. See the License for the specific language governing
+// permissions and limitations under the License.
+
 package crdbsqlx
 
 import (

crdb/crdbsqlx/sqlx_test.go

@@ -1,3 +1,17 @@
+// Copyright 2016 The Cockroach Authors.
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or
+// implied. See the License for the specific language governing
+// permissions and limitations under the License.
+
 package crdb
 
 import "fmt"

crdb/error.go

@@ -210,6 +210,9 @@ func (ts *testServerImpl) NewTenantServer(proxy bool) (TestServer, error) {
 	}
 
 	tenantDB, err := sql.Open("postgres", tenantURL.String())
+	if err != nil {
+		return nil, err
+	}
 	defer tenantDB.Close()
 
 	rootPassword := ""