DRYify input validation

Aperta
#452 1 commento 0 reazioni 0 assegnatari Vedi su GitHub

I maintainer di solito rispondono entro 2 giorni

Nessuno ha ancora preso questa issue.

Valutazione

Difficoltà
5/5
Tempo stimato
Più di una settimana
Idoneità per principianti
25/100
Tipo di issue
Refactoring
Chiarezza
Da chiarire
Stato di attività
Ferma
Stack tecnologico
python
Ambito
backend

Direzione di ricerca

Inizia dal codice di convalida degli input in pyrenew/process/differencedprocess.py, alle righe 234–245, e analizza i pattern di convalida nelle vicinanze. Identifica le attività comuni che dovrebbero essere generalizzate e valuta se è adatta una libreria diffusa e testata; il lavoro è completato quando l’approccio di convalida concordato è definito e l’esempio esistente è coperto.

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

Descrizione

Would be good to have generic functions for common input validation tasks. Even better to depend on a well-liked/tested library for this.

Example: checking for an integer > x, which happens in a non-generic way here: https://github.com/CDCgov/PyRenew/blob/afbb340ce35176c5fb4c23afc05bfa1ee777841e/pyrenew/process/differencedprocess.py#L234-L245

Lingua principale
Python
Stelle
30
Fork
11
Merge medio
5g 4h
PR unite (30g)
9

Preparare l'ambiente

Questo progetto non fornisce container di sviluppo, Dockerfile né guida per i contributori, quindi l'ambiente è a tuo carico: parti dal suo README e consulta la nostra guida al primo contributo per i passaggi generali.

Come iniziare

  1. Leggi tutta la issue e poi la guida ai contributi del progetto.
  2. Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
  3. Fai un fork del repository e lavora su un branch.
  4. Apri una pull request che faccia riferimento al numero della issue.

Altre issue di CDCgov/PyRenew

Tutte le issue di CDCgov/PyRenew

Issue simili

Altre issue su Python

Ricevi le nuove issue nella tua casella

Un breve riepilogo di issue GitHub adatte ai principianti.