livepeer/go-livepeer

Run golangci-lint on all packages

Aperta

#1801 aperta il 18 mar 2021

Vedi su GitHubGuida contributor
 (2 commenti) (0 reazioni) (0 assegnatari)Go (223 fork)auto 404
area: CIgood first issuetype: tech debt

Metriche repository

Star
 (583 stelle)
Metriche merge PR
 (Metriche PR in attesa)

Descrizione

We currently only run golangci-lint on certain packages in CI. We should run golangci-lint locally, fix all the warnings/errors and then start running golangci-lint on all packages in CI. The one thing to consider here is which linters should be enabled by default (or just use the default linters used by golangci-lint).

Guida contributor

Direzione di ricerca
Esegui golangci lint localmente su tutti i pacchetti, correggi tutti gli avvisi e gli errori, quindi aggiorna il file del flusso di lavoro CI per eseguire golangci lint su tutti i pacchetti invece che solo su quelli specifici.
Tech stack
go
Dominio
build systemdevops
Tipo issue
Manutenzione
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Attiva
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Chiara
Prerequisiti
Gogolangci lint
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
70