Skip to content

Commit f7f6827

Browse files
committed
fix: adjusting linter config, fixes
1 parent 4eef1b5 commit f7f6827

File tree

15 files changed

+87
-48
lines changed

15 files changed

+87
-48
lines changed

.gitignore

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,4 @@
11
.idea/
22
cp.out
33
.DS_Store
4+
.bin

.golangci.yml

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -26,10 +26,10 @@ linters-settings:
2626
gocyclo:
2727
min-complexity: 15
2828
gofumpt:
29-
module-path: github.com/snyk/cli-extension-os-flows
29+
module-path: github.com/snyk/cli-extension-dep-graph
3030
extra-rules: true
3131
goimports:
32-
local-prefixes: github.com/snyk/cli-extension-os-flows
32+
local-prefixes: github.com/snyk/cli-extension-dep-graph
3333
gosimple:
3434
checks: ["all"]
3535
govet:
@@ -53,7 +53,6 @@ linters-settings:
5353
revive:
5454
rules:
5555
- name: blank-imports
56-
disabled: true
5756
- name: context-as-argument
5857
- name: context-keys-type
5958
- name: dot-imports
@@ -65,7 +64,6 @@ linters-settings:
6564
- name: increment-decrement
6665
- name: var-naming
6766
- name: var-declaration
68-
disabled: true
6967
- name: package-comments
7068
- name: range
7169
- name: receiver-naming
@@ -78,6 +76,7 @@ linters-settings:
7876
- name: unused-parameter
7977
- name: unreachable-code
8078
- name: redefines-builtin-id
79+
- name: redundant-import-alias
8180
staticcheck:
8281
checks: ["all"]
8382
stylecheck:

Makefile

Lines changed: 12 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,25 @@ GOMOD=$(GOCMD) mod
55
GOBUILD=$(GOCMD) build
66
GOTEST=$(GOCMD) test
77

8+
GOLANGCI_LINT_V=v1.64.6
9+
810
all:
911
$(info "completed running make file for golang project")
12+
1013
fmt:
1114
@go fmt ./...
15+
16+
install-tools:
17+
@echo "Installing golangci-lint..."
18+
@mkdir -p .bin
19+
@curl -sSfL https://raw.githubusercontent.com/golangci/golangci-lint/master/install.sh | sh -s -- -b .bin $(GOLANGCI_LINT_V)
20+
1221
lint:
13-
./script/lint.sh
22+
$(if $(CI),golangci-lint,.bin/golangci-lint) run -v
23+
1424
tidy:
1525
$(GOMOD) tidy -v
26+
1627
test:
1728
$(GOTEST) ./... -coverprofile cp.out
1829

go.mod

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,7 @@ require (
4242
github.com/golang/groupcache v0.0.0-20241129210726-2c02b8208cf8 // indirect
4343
github.com/google/uuid v1.6.0 // indirect
4444
github.com/hashicorp/go-uuid v1.0.3 // indirect
45+
github.com/inconshreveable/mousetrap v1.1.0 // indirect
4546
github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 // indirect
4647
github.com/jcmturner/aescts/v2 v2.0.0 // indirect
4748
github.com/jcmturner/dnsutils/v2 v2.0.0 // indirect
@@ -62,6 +63,7 @@ require (
6263
github.com/pkg/browser v0.0.0-20210911075715-681adbf594b8 // indirect
6364
github.com/pkg/errors v0.9.1 // indirect
6465
github.com/pmezard/go-difflib v1.0.1-0.20181226105442-5d4384ee4fb2 // indirect
66+
github.com/puzpuzpuz/xsync v1.5.2 // indirect
6567
github.com/rivo/uniseg v0.4.7 // indirect
6668
github.com/sabhiram/go-gitignore v0.0.0-20210923224102-525f6e181f06 // indirect
6769
github.com/sagikazarmark/locafero v0.7.0 // indirect
@@ -72,9 +74,13 @@ require (
7274
github.com/sourcegraph/conc v0.3.0 // indirect
7375
github.com/spf13/afero v1.14.0 // indirect
7476
github.com/spf13/cast v1.7.1 // indirect
77+
github.com/spf13/cobra v1.9.1 // indirect
7578
github.com/spf13/viper v1.20.1 // indirect
7679
github.com/subosito/gotenv v1.6.0 // indirect
7780
github.com/xanzy/ssh-agent v0.3.3 // indirect
81+
github.com/xeipuuv/gojsonpointer v0.0.0-20180127040702-4e3ac2762d5f // indirect
82+
github.com/xeipuuv/gojsonreference v0.0.0-20180127040603-bd5ef7bd5415 // indirect
83+
github.com/xeipuuv/gojsonschema v1.2.0 // indirect
7884
github.com/xo/terminfo v0.0.0-20220910002029-abceb7e1c41e // indirect
7985
go.uber.org/atomic v1.9.0 // indirect
8086
go.uber.org/multierr v1.9.0 // indirect

go.sum

Lines changed: 20 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,6 +32,7 @@ github.com/charmbracelet/x/term v0.2.1/go.mod h1:oQ4enTYFV7QN4m0i9mzHrViD7TQKvNE
3232
github.com/cloudflare/circl v1.6.0 h1:cr5JKic4HI+LkINy2lg3W2jF8sHCVTBncJr5gIIq7qk=
3333
github.com/cloudflare/circl v1.6.0/go.mod h1:uddAzsPgqdMAYatqJ0lsjX1oECcQLIlRpzZh3pJrofs=
3434
github.com/coreos/go-systemd/v22 v22.5.0/go.mod h1:Y58oyj3AT4RCenI/lSvhwexgC+NSVTIJ3seZv2GcEnc=
35+
github.com/cpuguy83/go-md2man/v2 v2.0.6/go.mod h1:oOW0eioCTA6cOiMLiUPZOpcVxMig6NIQQ7OS05n1F4g=
3536
github.com/cyphar/filepath-securejoin v0.4.1 h1:JyxxyPEaktOD+GAnqIqTf9A8tHyAG22rowi7HkoSU1s=
3637
github.com/cyphar/filepath-securejoin v0.4.1/go.mod h1:Sdj7gXlvMcPZsbhwhQ33GguGLDGQL7h7bg04C/+u9jI=
3738
github.com/davecgh/go-spew v1.1.0/go.mod h1:J7Y8YcW2NihsgmVo/mv3lAwl/skON4iLHjSsI+c5H38=
@@ -84,6 +85,12 @@ github.com/gorilla/sessions v1.2.1/go.mod h1:dk2InVEVJ0sfLlnXv9EAgkf6ecYs/i80K/z
8485
github.com/hashicorp/go-uuid v1.0.2/go.mod h1:6SBZvOh/SIDV7/2o3Jml5SYk/TvGqwFJ/bN7x4byOro=
8586
github.com/hashicorp/go-uuid v1.0.3 h1:2gKiV6YVmrJ1i2CKKa9obLvRieoRGviZFL26PcT/Co8=
8687
github.com/hashicorp/go-uuid v1.0.3/go.mod h1:6SBZvOh/SIDV7/2o3Jml5SYk/TvGqwFJ/bN7x4byOro=
88+
github.com/hashicorp/go-version v1.7.0 h1:5tqGy27NaOTB8yJKUZELlFAS/LTKJkrmONwQKeRZfjY=
89+
github.com/hashicorp/go-version v1.7.0/go.mod h1:fltr4n8CU8Ke44wwGCBoEymUuxUHl09ZGVZPK5anwXA=
90+
github.com/hashicorp/logutils v1.0.0 h1:dLEQVugN8vlakKOUE3ihGLTZJRB4j+M2cdTm/ORI65Y=
91+
github.com/hashicorp/logutils v1.0.0/go.mod h1:QIAnNjmIWmVIIkWDTG1z5v++HQmx9WQRO+LraFDTW64=
92+
github.com/inconshreveable/mousetrap v1.1.0 h1:wN+x4NVGpMsO7ErUn/mUI3vEoE6Jt13X2s0bqwp9tc8=
93+
github.com/inconshreveable/mousetrap v1.1.0/go.mod h1:vpF70FUmC8bwa3OWnCshd2FqLfsEA9PFc4w1p2J65bw=
8794
github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99 h1:BQSFePA1RWJOlocH6Fxy8MmwDt+yVQYULKfN0RoTN8A=
8895
github.com/jbenet/go-context v0.0.0-20150711004518-d14ea06fba99/go.mod h1:1lJo3i6rXxKeerYnT8Nvf0QmHCRC1n8sfWVwXF2Frvo=
8996
github.com/jcmturner/aescts/v2 v2.0.0 h1:9YKLH6ey7H4eDBXW8khjYslgyqG2xZikXP0EQFKrle8=
@@ -127,6 +134,8 @@ github.com/oapi-codegen/runtime v1.1.1 h1:EXLHh0DXIJnWhdRPN2w4MXAzFyE4CskzhNLUmt
127134
github.com/oapi-codegen/runtime v1.1.1/go.mod h1:SK9X900oXmPWilYR5/WKPzt3Kqxn/uS/+lbpREv+eCg=
128135
github.com/onsi/gomega v1.34.1 h1:EUMJIKUjM8sKjYbtxQI9A4z2o+rruxnzNvpknOXie6k=
129136
github.com/onsi/gomega v1.34.1/go.mod h1:kU1QgUvBDLXBJq618Xvm2LUX6rSAfRaFRTcdOeDLwwY=
137+
github.com/pact-foundation/pact-go/v2 v2.4.1 h1:eaLC58qzeCTbwdlCY8UvWz1HmDW+qrjTFfH8Xoq0rWs=
138+
github.com/pact-foundation/pact-go/v2 v2.4.1/go.mod h1:OwnXXRliPZvKDMJn/IsAwQ95tQprmp5gPTzPYz54mTg=
130139
github.com/patrickmn/go-cache v2.1.0+incompatible h1:HRMgzkcYKYpi3C8ajMPV8OFXaaRUnok+kx1WdO15EQc=
131140
github.com/patrickmn/go-cache v2.1.0+incompatible/go.mod h1:3Qf8kWWT7OJRJbdiICTKqZju1ZixQ/KpMGzzAfe6+WQ=
132141
github.com/pelletier/go-toml/v2 v2.2.3 h1:YmeHyLY8mFWbdkNWwpr+qIL2bEqT0o95WSdkNHvL12M=
@@ -150,6 +159,7 @@ github.com/rogpeppe/go-internal v1.14.1/go.mod h1:MaRKkUm5W0goXpeCfT7UZI6fk/L7L7
150159
github.com/rs/xid v1.6.0/go.mod h1:7XoLgs4eV+QndskICGsho+ADou8ySMSjJKDIan90Nz0=
151160
github.com/rs/zerolog v1.34.0 h1:k43nTLIwcTVQAncfCw4KZ2VY6ukYoZaBPNOE8txlOeY=
152161
github.com/rs/zerolog v1.34.0/go.mod h1:bJsvje4Z08ROH4Nhs5iH600c3IkWhwp44iRc54W6wYQ=
162+
github.com/russross/blackfriday/v2 v2.1.0/go.mod h1:+Rmxgy9KzJVeS9/2gXHxylqXiyQDYRxCVz55jmeOWTM=
153163
github.com/sabhiram/go-gitignore v0.0.0-20210923224102-525f6e181f06 h1:OkMGxebDjyw0ULyrTYWeN0UNCCkmCWfjPnIA2W6oviI=
154164
github.com/sabhiram/go-gitignore v0.0.0-20210923224102-525f6e181f06/go.mod h1:+ePHsJ1keEjQtpvf9HHw0f4ZeJ0TLRsxhunSI2hYJSs=
155165
github.com/sagikazarmark/locafero v0.7.0 h1:5MqpDsTGNDhY8sGp0Aowyf0qKsPrhewaLSsFaodPcyo=
@@ -175,13 +185,17 @@ github.com/spf13/afero v1.14.0 h1:9tH6MapGnn/j0eb0yIXiLjERO8RB6xIVZRDCX7PtqWA=
175185
github.com/spf13/afero v1.14.0/go.mod h1:acJQ8t0ohCGuMN3O+Pv0V0hgMxNYDlvdk+VTfyZmbYo=
176186
github.com/spf13/cast v1.7.1 h1:cuNEagBQEHWN1FnbGEjCXL2szYEXqfJPbP2HNUaca9Y=
177187
github.com/spf13/cast v1.7.1/go.mod h1:ancEpBxwJDODSW/UG4rDrAqiKolqNNh2DX3mk86cAdo=
188+
github.com/spf13/cobra v1.9.1 h1:CXSaggrXdbHK9CF+8ywj8Amf7PBRmPCOJugH954Nnlo=
189+
github.com/spf13/cobra v1.9.1/go.mod h1:nDyEzZ8ogv936Cinf6g1RU9MRY64Ir93oCnqb9wxYW0=
178190
github.com/spf13/pflag v1.0.6 h1:jFzHGLGAlb3ruxLB8MhbI6A8+AQX/2eW4qeyNZXNp2o=
179191
github.com/spf13/pflag v1.0.6/go.mod h1:McXfInJRrz4CZXVZOBLb0bTZqETkiAhM9Iw0y3An2Bg=
180192
github.com/spf13/viper v1.20.1 h1:ZMi+z/lvLyPSCoNtFCpqjy0S4kPbirhpTMwl8BkW9X4=
181193
github.com/spf13/viper v1.20.1/go.mod h1:P9Mdzt1zoHIG8m2eZQinpiBjo6kCmZSKBClNNqjJvu4=
182194
github.com/spkg/bom v0.0.0-20160624110644-59b7046e48ad/go.mod h1:qLr4V1qq6nMqFKkMo8ZTx3f+BZEkzsRUY10Xsm2mwU0=
183195
github.com/stretchr/objx v0.1.0/go.mod h1:HFkY916IF+rwdDfMAkV7OtwuqBVzrE8GR6GFx+wExME=
184196
github.com/stretchr/objx v0.4.0/go.mod h1:YvHI0jy2hoMjB+UWwv71VJQ9isScKT/TqJzVSSt89Yw=
197+
github.com/stretchr/objx v0.5.2 h1:xuMeJ0Sdp5ZMRXx/aWO6RZxdr3beISkG5/G/aIRr3pY=
198+
github.com/stretchr/objx v0.5.2/go.mod h1:FRsXN1f5AsAjCGJKqEizvkpNtU+EGNCLh3NxZ/8L+MA=
185199
github.com/stretchr/testify v1.2.2/go.mod h1:a8OnRcib4nhh0OaRAV+Yts87kKdq0PP7pXfy6kDkUVs=
186200
github.com/stretchr/testify v1.3.0/go.mod h1:M5WIy9Dh21IEIfnGCwXGc5bZfKNJtfHm1UVUgZn+9EI=
187201
github.com/stretchr/testify v1.4.0/go.mod h1:j7eGeouHqKxXV5pUuKE4zz7dFj8WfuZ+81PSLYec5m4=
@@ -271,6 +285,12 @@ golang.org/x/tools v0.1.1/go.mod h1:o0xws9oXOQQZyjljx8fwUC0k7L1pTE6eaCbjGeHmOkk=
271285
golang.org/x/xerrors v0.0.0-20190717185122-a985d3407aa7/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
272286
golang.org/x/xerrors v0.0.0-20191011141410-1b5146add898/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
273287
golang.org/x/xerrors v0.0.0-20200804184101-5ec99f83aff1/go.mod h1:I/5z698sn9Ka8TeJc9MKroUUfqBBauWjQqLJ2OPfmY0=
288+
google.golang.org/genproto/googleapis/rpc v0.0.0-20250404141209-ee84b53bf3d0 h1:0K7wTWyzxZ7J+L47+LbFogJW1nn/gnnMCN0vGXNYtTI=
289+
google.golang.org/genproto/googleapis/rpc v0.0.0-20250404141209-ee84b53bf3d0/go.mod h1:qQ0YXyHHx3XkvlzUtpXDkS29lDSafHMZBAZDc03LQ3A=
290+
google.golang.org/grpc v1.71.0 h1:kF77BGdPTQ4/JZWMlb9VpJ5pa25aqvVqogsxNHHdeBg=
291+
google.golang.org/grpc v1.71.0/go.mod h1:H0GRtasmQOh9LkFoCPDu3ZrwUtD1YGE+b2vYBYd/8Ec=
292+
google.golang.org/protobuf v1.36.6 h1:z1NpPI8ku2WgiWnf+t9wTPsn6eP1L7ksHUlkfLvd9xY=
293+
google.golang.org/protobuf v1.36.6/go.mod h1:jduwjTPXsFjZGTmRluh+L6NjiWu7pchiJ2/5YcXBHnY=
274294
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
275295
gopkg.in/check.v1 v1.0.0-20190902080502-41f04d3bba15/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
276296
gopkg.in/check.v1 v1.0.0-20201130134442-10cb98267c6c h1:Hei/4ADfdWqJk1ZMxUNpqntNwaWcugrBjAiHlqqRiVk=

internal/uv/uv_plugin.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,12 @@ import (
77
"path/filepath"
88

99
"github.com/rs/zerolog"
10+
"github.com/snyk/dep-graph/go/pkg/depgraph"
11+
1012
"github.com/snyk/cli-extension-dep-graph/internal/conversion"
1113
"github.com/snyk/cli-extension-dep-graph/internal/snykclient"
1214
"github.com/snyk/cli-extension-dep-graph/pkg/ecosystems/discovery"
13-
scaplugin "github.com/snyk/cli-extension-dep-graph/pkg/sca_plugin"
14-
"github.com/snyk/dep-graph/go/pkg/depgraph"
15+
"github.com/snyk/cli-extension-dep-graph/pkg/scaplugin"
1516
)
1617

1718
type Plugin struct {
@@ -193,4 +194,4 @@ func (p Plugin) discoverLockFiles(
193194
return files, nil
194195
}
195196

196-
var _ scaplugin.ScaPlugin = (*Plugin)(nil)
197+
var _ scaplugin.SCAPlugin = (*Plugin)(nil)

internal/uv/uv_plugin_test.go

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -10,12 +10,13 @@ import (
1010
"testing"
1111

1212
"github.com/rs/zerolog"
13-
"github.com/snyk/cli-extension-dep-graph/internal/mocks"
14-
"github.com/snyk/cli-extension-dep-graph/internal/snykclient"
15-
scaplugin "github.com/snyk/cli-extension-dep-graph/pkg/sca_plugin"
1613
"github.com/snyk/dep-graph/go/pkg/depgraph"
1714
"github.com/stretchr/testify/assert"
1815
"github.com/stretchr/testify/require"
16+
17+
"github.com/snyk/cli-extension-dep-graph/internal/mocks"
18+
"github.com/snyk/cli-extension-dep-graph/internal/snykclient"
19+
"github.com/snyk/cli-extension-dep-graph/pkg/scaplugin"
1920
)
2021

2122
var logger = zerolog.Nop()

internal/uv/uvclient.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,11 @@ import (
88
"strconv"
99
"strings"
1010

11-
scaplugin "github.com/snyk/cli-extension-dep-graph/pkg/sca_plugin"
1211
clierrors "github.com/snyk/error-catalog-golang-public/cli"
1312
"github.com/snyk/error-catalog-golang-public/opensource/ecosystems"
1413
"github.com/snyk/error-catalog-golang-public/snyk_errors"
14+
15+
"github.com/snyk/cli-extension-dep-graph/pkg/scaplugin"
1516
)
1617

1718
const (

internal/uv/uvclient_test.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,10 +4,11 @@ import (
44
"errors"
55
"testing"
66

7-
scaplugin "github.com/snyk/cli-extension-dep-graph/pkg/sca_plugin"
87
"github.com/snyk/error-catalog-golang-public/snyk_errors"
98
"github.com/stretchr/testify/assert"
109
"github.com/stretchr/testify/require"
10+
11+
"github.com/snyk/cli-extension-dep-graph/pkg/scaplugin"
1112
)
1213

1314
type mockCmdExecutor struct {

pkg/depgraph/callback.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,8 +1,6 @@
11
package depgraph
22

33
import (
4-
_ "embed"
5-
64
"github.com/rs/zerolog"
75
"github.com/snyk/go-application-framework/pkg/configuration"
86
"github.com/snyk/go-application-framework/pkg/workflow"

0 commit comments

Comments
 (0)