-
Notifications
You must be signed in to change notification settings - Fork 1
Review #39
Copy link
Copy link
Open
Description
- utilizzati conventional commits
- 32 PRs con annessa pipeline ed occasionali code-review, le code-review potevano essere un po' di più
- buon README
- il bot sembra funzionare abbastanza bene però ogni tanto invia un errore nel terminale con
KeyError: 'date' - 100% code coverage 🚀 🎖️
- minor: ottimo .gitignore
- codice tipizzato
il progetto mi sembra molto originale e ben curato nei dettagli, anche tecnicamente, sicuramente si può sempre migliorare, però mi sembra molto "completo" come progetto, inoltre considerando che siete in due avete fatto un buon lavoro, bravi 💪
Piccoli suggerimenti:
- le MAGIC string/numbers vanno ridotte al minimo, sempre (es. in durata_timer.py e load_file.py ci sono tante magic strings in comune, si potrebbero convetire in costanti condivise tra i file per evitare la ridondanza e migliorare la riusabilità)
- le PR sono poche commentate probabilmente per la poco livello di conoscenza del linguaggio, ci sono molte cose simpatiche da utilizzare che abbiamo visto anche a QD come l'unpack (o destructuring)
# senza unpack
args = call_context.job.context
update = args[0]
rm_id = args[1]
context = args[2]
# usando la feature unpack
update, rm_id, context = call_context.job.context- minor: la funzione
init()sumain.pyforse non era necessaria
Comunque, bravissimi 🚀
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels