Adopt ruff linting (would have caught the un-raised-exception pattern)

Aperta

#791 aperta il 6/7/2026 0 commenti 0 reazioni 0 assegnatari Vedi su GitHub

Una pull request collegata è già stata integrata.

#818 di @cailmdaley — integrata

Valutazione

Difficoltà: 4/5
Tempo stimato: 3-5 giorni
Idoneità per principianti: 55/100

Tipo di issue: Funzionalità
Chiarezza: Abbastanza chiara
Stato di attività: Tranquilla
Stack tecnologico: python
Ambito: ci-cd, tooling

Direzione di ricerca

Inizia esaminando la configurazione esistente delle dipendenze di sviluppo e della CI del repository, quindi esamina i cinque file menzionati: psfex_interp_runner.py, mccd_interp_runner.py, mask_package/mask.py, pipeline/file_handler.py e pipeline/file_io.py. Il lavoro è completato quando Ruff è stato aggiunto con le regole iniziali proposte e la CI impone le nuove violazioni senza richiedere una pulizia immediata del codice esistente.

Scritto dal modello di indicizzazione a partire dal testo della issue.

Descrizione

Motivation

PR #789 (fixing #785) found the same bug — an exception constructed but never raised, silently falling through an error path — at five independent sites across the codebase (psfex_interp_runner.py, mccd_interp_runner.py ×2, mask_package/mask.py, pipeline/file_handler.py, pipeline/file_io.py). Ruff's PLW0133 (useless-exception-statement) flags exactly this pattern; a linter in CI would have caught all five at introduction time.

Proposal

Add ruff to the dev tooling with a conservative initial rule set (e.g. E, F, PLW0133 and friends) — start narrow, expand as the codebase is cleaned.
Wire it into CI so new violations block, without requiring a big-bang cleanup of existing code (per-file ignores or --diff-scoped enforcement).

Sibling precedent: sp_validation has a project-wide ruff pass planned (CosmoStat/sp_validation#204).

— Claude (Fable) on behalf of Cail

Lingua principale: Python
Stelle: 18
Fork: 14
Merge medio: 8h 40m
PR unite (30g): 10

Guida per i contributori

Apri la 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.