Skip to content

Commit

Permalink
Remove deadcode checker CI (#472)
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
bschimke95 authored Jun 6, 2024
1 parent 910305d commit b7859b0
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 b7859b0

Please sign in to comment.