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
- 47
- Merge medio
- 2g 19h
- PR unite (30g)
- 32
Preparare l'ambiente
Non abbiamo ancora controllato i file di configurazione di questo progetto. Parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.
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
-
enhancement go
Difficoltà 4/5 3-5 giorni Idoneità per principianti 62/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 58/100
I maintainer di solito rispondono entro 1 giorno
-
authz bug
Difficoltà 3/5 1-2 giorni Idoneità per principianti 70/100
raystack/frontier#1896 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Add OpenTelemetry tracingAperta
Difficoltà 5/5 Più di una settimana Idoneità per principianti 45/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 5/5 Più di una settimana Idoneità per principianti 38/100
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di raystack/frontier
Issue simili
-
agentic-workflows
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno
-
priority/4/normal status/needs-triage type/bug/unconfirmed
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
authelia/authelia#13292 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
blinklabs-io/actions#138 ·
I maintainer di solito rispondono entro 1 giorno
-
[UI] AlbumDetails collapses multi-genre list to single primary genre on viewports < lg breakpointAperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
I maintainer di solito rispondono entro 1 giorno