proposal: add more linters and fix existing issues
I maintainer di solito rispondono entro 1 giorno
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 35/100
Direzione di ricerca
Inizia individuando la configurazione della pipeline di linting del repository ed esegui i controlli esistenti per comprendere la configurazione attuale. Aggiungi gocognit, gosec, staticcheck ed errcheck, estendi gosec oltre G201 e G202, quindi risolvi i problemi segnalati dai controlli abilitati.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Summary
Add the following linters to improve code quality, security, and maintainability:
- gocognit
- gosec
- staticcheck
- errcheck
Currently, gosec is only using rules G201 and G202. Existing issues from other relevant rules also need to be identified and fixed.
Why this is useful
Adding these linters will help catch issues earlier in development and reduce the chances of bugs or security problems reaching production.
gocognit
- Detects functions with high cognitive complexity
- Helps keep code easier to read, review, test, and maintain
- Encourages simpler control flow and clearer logic
gosec
- Detects insecure coding patterns
- Helps identify issues beyond
G201andG202 - Improves security by catching risky patterns early in CI and local development
staticcheck
- Finds correctness, performance, and code quality issues
- Detects ineffective assignments, broken assumptions, deprecated usage, and other subtle problems
- Improves overall reliability of the codebase
errcheck
- Ensures returned errors are not ignored
- Helps prevent silent failures and inconsistent behavior
- Encourages explicit and reliable error handling
Expected benefits
- Catch bugs earlier in CI
- Improve security posture
- Make the codebase easier to maintain and refactor
- Standardize code quality checks across the repository
Scope
- Add the new linters to the linting pipeline
- Expand
goseccoverage beyondG201andG202 - Fix the issues reported by the newly enabled checks
Notes
These linters will help make the code more secure and less complex to read.
- Lingua principale
- Go
- Stelle
- 344
- Fork
- 48
- Merge medio
- 2g 9h
- PR unite (30g)
- 47
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di raystack/frontier
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 3/5 1-2 giorni Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 52/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di raystack/frontier
Issue simili
-
component/gpu-kubelet-plugin component/helm needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
kubernetes-sigs/dra-driver-nvidia-gpu#1559 ·
I maintainer di solito rispondono entro 1 giorno
-
TOTP enrollment returns 500 for phone-only users (empty TOTP account name)Forse già presa @emiliosandoval4 l’ha presa oggi. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 60/100
I maintainer di solito rispondono entro 1 giorno
-
8.9 to 8.10 Helm upgrade guide still says deprecated keys are removed in chart v16 (Camunda 8.11)Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 78/100
camunda/camunda-platform-helm#7559 ·
I maintainer di solito rispondono entro 1 giorno
-
docs: namespaceRegexes example teaches prefix matching, but regexes are matched as a substringForse già presa Una pull request collegata a questa issue è aperta o già unita. Apertatriage/pending-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 74/100
external-secrets/external-secrets#7117 ·
I maintainer di solito rispondono entro 2 giorni
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 72/100
ENTERPILOT/GoModel#1183 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno