Skip to content

Commit

Permalink
Merge pull request #252 from maxmind/horgh/lint
Browse files Browse the repository at this point in the history
Fix lint
  • Loading branch information
faktas2 authored Aug 28, 2023
2 parents 073b56b + 1d66744 commit a620550
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 3 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/go.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ jobs:
build:
strategy:
matrix:
go-version: [1.19.x, 1.20.x]
go-version: [1.20.x, 1.21.x]
platform: [ubuntu-latest, macos-latest, windows-latest]
runs-on: ${{ matrix.platform }}
name: "Build ${{ matrix.go-version }} test on ${{ matrix.platform }}"
Expand Down
2 changes: 1 addition & 1 deletion README.md
Original file line number Diff line number Diff line change
Expand Up @@ -102,7 +102,7 @@ Please see our [Docker documentation](doc/docker.md).

### Installation from source or Git

You need the Go compiler (1.19+). You can get it at the [Go
You need the Go compiler (1.20+). You can get it at the [Go
website](https://golang.org).

The easiest way is via `go install`:
Expand Down
3 changes: 2 additions & 1 deletion pkg/geoipupdate/config_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -828,7 +828,8 @@ func TestValidateConfig(t *testing.T) {

for _, test := range tests {
t.Run(test.Description, func(t *testing.T) {
err := validateConfig(&test.Config)
config := test.Config
err := validateConfig(&config)
if test.Err == "" {
assert.NoError(t, err, test.Description)
} else {
Expand Down

0 comments on commit a620550

Please sign in to comment.