Skip to content

Commit

Permalink
Remove deadcode checker
Browse files Browse the repository at this point in the history
The deadcode checker doesn't work well with our features setup as it
detects non-main feature modules (that are used e.g. in moonray) as unused
and blocks the CI.
That checker hasn't really contributed anything useful anyway, so let's
just remove it.
  • Loading branch information
k8s-bot committed Jun 5, 2024
1 parent 07eca11 commit bd7f211
Show file tree
Hide file tree
Showing 6 changed files with 1 addition and 43 deletions.
4 changes: 0 additions & 4 deletions .github/workflows/go.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -61,10 +61,6 @@ jobs:
working-directory: src/k8s
run: make go.vet

- name: go deadcode
working-directory: src/k8s
run: make go.deadcode

- name: go test
working-directory: src/k8s
run: make go.unit
Expand Down
3 changes: 0 additions & 3 deletions src/k8s/Makefile
Original file line number Diff line number Diff line change
Expand Up @@ -13,9 +13,6 @@ go.vet:
go.unit:
$(DQLITE_BUILD_SCRIPTS_DIR)/static-go-test.sh -v ./pkg/... ./cmd/... ./api/... -coverprofile=coverage.txt --cover

go.deadcode:
$(TOOLS_DIR)/deadcode.sh

go.doc: bin/static/k8s
bin/static/k8s generate-docs --output-dir ../../docs/src/_parts/commands/

Expand Down
20 changes: 0 additions & 20 deletions src/k8s/tools/deadcode.sh

This file was deleted.

8 changes: 1 addition & 7 deletions src/k8s/tools/go.mod
Original file line number Diff line number Diff line change
Expand Up @@ -2,10 +2,7 @@ module github.com/canonical/k8s/tools

go 1.22

require (
github.com/regclient/regclient v0.6.1
golang.org/x/tools v0.20.0
)
require github.com/regclient/regclient v0.6.1

require (
github.com/docker/libtrust v0.0.0-20160708172513-aabc10ec26b7 // indirect
Expand All @@ -17,9 +14,6 @@ require (
github.com/spf13/cobra v1.8.0 // indirect
github.com/spf13/pflag v1.0.5 // indirect
github.com/ulikunitz/xz v0.5.12 // indirect
golang.org/x/mod v0.17.0 // indirect
golang.org/x/sync v0.7.0 // indirect
golang.org/x/sys v0.20.0 // indirect
golang.org/x/telemetry v0.0.0-20240405141642-5e7a33625e65 // indirect
gopkg.in/yaml.v3 v3.0.1 // indirect
)
8 changes: 0 additions & 8 deletions src/k8s/tools/go.sum
Original file line number Diff line number Diff line change
Expand Up @@ -30,17 +30,9 @@ github.com/stretchr/testify v1.7.0 h1:nwc3DEeHmmLAfoZucVR881uASk0Mfjw8xYJ99tb5Cc
github.com/stretchr/testify v1.7.0/go.mod h1:6Fq8oRcR53rry900zMqJjRRixrwX3KX962/h/Wwjteg=
github.com/ulikunitz/xz v0.5.12 h1:37Nm15o69RwBkXM0J6A5OlE67RZTfzUxTj8fB3dfcsc=
github.com/ulikunitz/xz v0.5.12/go.mod h1:nbz6k7qbPmH4IRqmfOplQw/tblSgqTqBwxkY0oWt/14=
golang.org/x/mod v0.17.0 h1:zY54UmvipHiNd+pm+m0x9KhZ9hl1/7QNMyxXbc6ICqA=
golang.org/x/mod v0.17.0/go.mod h1:hTbmBsO62+eylJbnUtE2MGJUyE7QWk4xUqPFrRgJ+7c=
golang.org/x/sync v0.7.0 h1:YsImfSBoP9QPYL0xyKJPq0gcaJdG3rInoqxTWbfQu9M=
golang.org/x/sync v0.7.0/go.mod h1:Czt+wKu1gCyEFDUtn0jG5QVvpJ6rzVqr5aXyt9drQfk=
golang.org/x/sys v0.0.0-20220715151400-c0bba94af5f8/go.mod h1:oPkhp1MJrh7nUepCBck5+mAzfO9JrbApNNgaTdGDITg=
golang.org/x/sys v0.20.0 h1:Od9JTbYCk261bKm4M/mw7AklTlFYIa0bIp9BgSm1S8Y=
golang.org/x/sys v0.20.0/go.mod h1:/VUhepiaJMQUp4+oa/7Zr1D23ma6VTLIYjOOTFZPUcA=
golang.org/x/telemetry v0.0.0-20240405141642-5e7a33625e65 h1:2YD4MqNQYEQlI4t931POAM5w8fe4ZFkHRWTJhRIAjz8=
golang.org/x/telemetry v0.0.0-20240405141642-5e7a33625e65/go.mod h1:W6pterlIx28wS98MzYROSpZ8u12XJW5a0ZVeaJ6pfSo=
golang.org/x/tools v0.20.0 h1:hz/CVckiOxybQvFw6h7b/q80NTr9IUQb4s1IIzW7KNY=
golang.org/x/tools v0.20.0/go.mod h1:WvitBU7JJf6A4jOdg4S1tviW9bhUxkgeCui/0JHctQg=
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405 h1:yhCVgyC4o1eVCa2tZl7eS0r+SDo693bJlVdllGtEeKM=
gopkg.in/check.v1 v0.0.0-20161208181325-20d25e280405/go.mod h1:Co6ibVJAznAaIkqp8huTwlJQCZ016jof/cbN4VW5Yz0=
gopkg.in/yaml.v3 v3.0.0-20200313102051-9f266ea9e77c/go.mod h1:K4uyk7z7BCEPqu6E+C64Yfv1cQ7kz7rIZviUmN+EgEM=
Expand Down
1 change: 0 additions & 1 deletion src/k8s/tools/tools.go
Original file line number Diff line number Diff line change
Expand Up @@ -2,5 +2,4 @@ package main

import (
_ "github.com/regclient/regclient/cmd/regsync"
_ "golang.org/x/tools/cmd/deadcode"
)

0 comments on commit bd7f211

Please sign in to comment.