Skip to content

Commit ffe7ed3

Browse files
author
Evan Lezar
committed
Merge branch 'goimports-local' into 'main'
run goimports -local against the entire codebase See merge request nvidia/container-toolkit/container-toolkit!512
2 parents 5c78e2b + 7627d48 commit ffe7ed3

File tree

93 files changed

+215
-115
lines changed

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

93 files changed

+215
-115
lines changed

.golangci.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -15,6 +15,10 @@ linters:
1515
- staticcheck
1616
- unconvert
1717

18+
linters-settings:
19+
goimports:
20+
local-prefixes: github.com/NVIDIA/nvidia-container-toolkit
21+
1822
issues:
1923
exclude-rules:
2024
# Exclude the gocritic dupSubExpr issue for cgo files.

Makefile

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -78,6 +78,11 @@ fmt:
7878
go list -f '{{.Dir}}' $(MODULE)/... \
7979
| xargs gofmt -s -l -w
8080

81+
# Apply goimports -local github.com/NVIDIA/container-toolkit to the codebase
82+
goimports:
83+
go list -f {{.Dir}} $(MODULE)/... \
84+
| xargs goimports -local $(MODULE) -w
85+
8186
golangci-lint:
8287
golangci-lint run ./...
8388

cmd/nvidia-container-runtime-hook/container_config.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,9 +9,10 @@ import (
99
"path/filepath"
1010
"strings"
1111

12-
"github.com/NVIDIA/nvidia-container-toolkit/internal/config/image"
1312
"github.com/opencontainers/runtime-spec/specs-go"
1413
"golang.org/x/mod/semver"
14+
15+
"github.com/NVIDIA/nvidia-container-toolkit/internal/config/image"
1516
)
1617

1718
const (

cmd/nvidia-container-runtime-hook/container_config_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@ import (
55
"path/filepath"
66
"testing"
77

8-
"github.com/NVIDIA/nvidia-container-toolkit/internal/config/image"
98
"github.com/stretchr/testify/require"
9+
10+
"github.com/NVIDIA/nvidia-container-toolkit/internal/config/image"
1011
)
1112

1213
func TestGetNvidiaConfig(t *testing.T) {

cmd/nvidia-container-runtime-hook/hook_config_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,8 +21,9 @@ import (
2121
"os"
2222
"testing"
2323

24-
"github.com/NVIDIA/nvidia-container-toolkit/internal/config/image"
2524
"github.com/stretchr/testify/require"
25+
26+
"github.com/NVIDIA/nvidia-container-toolkit/internal/config/image"
2627
)
2728

2829
func TestGetHookConfig(t *testing.T) {

cmd/nvidia-container-runtime/main_test.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -11,11 +11,12 @@ import (
1111
"strings"
1212
"testing"
1313

14-
"github.com/NVIDIA/nvidia-container-toolkit/internal/modifier"
15-
"github.com/NVIDIA/nvidia-container-toolkit/internal/test"
1614
"github.com/opencontainers/runtime-spec/specs-go"
1715
testlog "github.com/sirupsen/logrus/hooks/test"
1816
"github.com/stretchr/testify/require"
17+
18+
"github.com/NVIDIA/nvidia-container-toolkit/internal/modifier"
19+
"github.com/NVIDIA/nvidia-container-toolkit/internal/test"
1920
)
2021

2122
const (

cmd/nvidia-ctk/cdi/cdi.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,11 +17,12 @@
1717
package cdi
1818

1919
import (
20+
"github.com/urfave/cli/v2"
21+
2022
"github.com/NVIDIA/nvidia-container-toolkit/cmd/nvidia-ctk/cdi/generate"
2123
"github.com/NVIDIA/nvidia-container-toolkit/cmd/nvidia-ctk/cdi/list"
2224
"github.com/NVIDIA/nvidia-container-toolkit/cmd/nvidia-ctk/cdi/transform"
2325
"github.com/NVIDIA/nvidia-container-toolkit/internal/logger"
24-
"github.com/urfave/cli/v2"
2526
)
2627

2728
type command struct {

cmd/nvidia-ctk/cdi/generate/generate.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,15 @@ import (
2222
"path/filepath"
2323
"strings"
2424

25+
"github.com/urfave/cli/v2"
26+
cdi "tags.cncf.io/container-device-interface/pkg/parser"
27+
2528
"github.com/NVIDIA/nvidia-container-toolkit/internal/config"
2629
"github.com/NVIDIA/nvidia-container-toolkit/internal/logger"
2730
"github.com/NVIDIA/nvidia-container-toolkit/internal/platform-support/tegra/csv"
2831
"github.com/NVIDIA/nvidia-container-toolkit/pkg/nvcdi"
2932
"github.com/NVIDIA/nvidia-container-toolkit/pkg/nvcdi/spec"
3033
"github.com/NVIDIA/nvidia-container-toolkit/pkg/nvcdi/transform"
31-
"github.com/urfave/cli/v2"
32-
cdi "tags.cncf.io/container-device-interface/pkg/parser"
3334
)
3435

3536
const (

cmd/nvidia-ctk/cdi/list/list.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,9 +19,10 @@ package list
1919
import (
2020
"fmt"
2121

22-
"github.com/NVIDIA/nvidia-container-toolkit/internal/logger"
2322
"github.com/urfave/cli/v2"
2423
"tags.cncf.io/container-device-interface/pkg/cdi"
24+
25+
"github.com/NVIDIA/nvidia-container-toolkit/internal/logger"
2526
)
2627

2728
type command struct {

cmd/nvidia-ctk/cdi/transform/root/root.go

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -21,11 +21,12 @@ import (
2121
"io"
2222
"os"
2323

24+
"github.com/urfave/cli/v2"
25+
"tags.cncf.io/container-device-interface/pkg/cdi"
26+
2427
"github.com/NVIDIA/nvidia-container-toolkit/internal/logger"
2528
"github.com/NVIDIA/nvidia-container-toolkit/pkg/nvcdi/spec"
2629
transformroot "github.com/NVIDIA/nvidia-container-toolkit/pkg/nvcdi/transform/root"
27-
"github.com/urfave/cli/v2"
28-
"tags.cncf.io/container-device-interface/pkg/cdi"
2930
)
3031

3132
type command struct {

0 commit comments

Comments
 (0)