kubernetes-sigs/cluster-api

Lint unused funcs

Aberta

#7.599 aberto em 23 de nov. de 2022

Ver no GitHubGuia do colaborador
 (12 comentários) (0 reação) (1 responsável)Go (1.532 forks)auto 404
help wantedkind/featurepriority/backlogtriage/accepted

Métricas do repositório

Stars
 (4.267 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

Description

It would be great if we had a linter which detects unused funcs.

I recently opened a PR to drop some unused code in KCP: https://github.com/kubernetes-sigs/cluster-api/pull/7598

I found the following cases:

  1. funcs which weren't used by anyone: e.g. ControlPlane.MachineInfrastructureTemplateRef
  2. funcs which were only used by test code: e.g. ControlPlane.NewMachine

I think if there is a linter 1. should be relatively easy to detect.

I'm not sure about 2., it could be that there are a lot of false positives as we probably have non-test code which is used as test utils in tests.

But even if we only find a linter for 1. it should be a good improvement

/kind feature

Guia do colaborador

Direção de pesquisa
Explore linters Go existentes, como `unused`, ou escreva um personalizado para detectar funções não utilizadas no código, focando primeiro no código que não é de teste.
Pilha de tecnologia
go
Domain
developer experience
Tipo Issue
Funcionalidade
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
2
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Ativo
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
Go
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
70