Review - Githubissues

linuchs / Botquake

Botquake

0 stars 2 forks source link

Review #32

Open Helias opened 7 months ago

Helias commented 7 months ago

Feedback sul progetto:

PROs:

88% code coverage
buon README
test parametrizzati
conventional commits parzialmente rispettati
22 PRs merged 👍
nessun warning di pylint
secret token TELEGRAM_BOT

CONs:

nessun file requirements.txt ma solo requirements_dev.txt, in requirements_dev.txt dovrebbero esserci solo le dipendeze relative al lint ed ai test, non le librerie per avviare il bot (come python-telegram-bot)
le funzioni non sono tutte tipizzate

Non male! 👍

linuchs commented 7 months ago

Applicheremo questi ultimi suggerimenti. E' stata una bella opportunità per ampliare le nostre conoscenze, grazie!