livebud/bud

generated app directory fails linting with golangci-lint

Aberta

#331 aberto em 27 de nov. de 2022

Ver no GitHub
 (1 comentário) (0 reação) (0 responsável)JavaScript (211 forks)batch import
help wanted

Métricas do repositório

Stars
 (5.391 estrelas)
Métricas de merge de PR
 (Nenhuma PRs mesclada em 30d)

Description

golangci-lint run --fix
bud/internal/app/main.go:74:19: Error return value of `budClient.Publish` is not checked (errcheck)
	budClient.Publish("app:ready", nil)

As part of CI I run a linting of my repository. I suggest adding the line //nolint at the top of the template for bud/internal/app/main.go, as well as any other bud-managed template boilerplate files. This way any linters people use will ignore these files (without having to do special configuration to exclude the bud directory like is already automagically done in the .gitignore.)

Guia do colaborador

Direção de pesquisa
Adicione um comentário //nolint ao arquivo de modelo gerado bud/internal/app/main.go para suprimir o erro de lint errcheck.
Pilha de tecnologia
go
Domain
backend
Tipo Issue
Bug
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.
Menos de 1 hora
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.
85