-
Notifications
You must be signed in to change notification settings - Fork 1
Review #61
Copy link
Copy link
Open
Description
PROs
- 60 PRs, un po' tante ma meglio di poche
- codice abbastanza tipizzato
- conventional commits rispettati
- abbastanza unit tests
CONs
- ho notato che avete creato 10 PRs di seguito per creare più "tentativi" nel fixare la pipeline, usando propriamente git e le branch era possibile crearne una con una sola branch e verificare l'utilizzo della pipeline inviando più commit a poco a poco nella branch della relativa PR
- c'è un solo file bot.py, si poteva migliorare la struttura del progetto dividendo il file bot.py in più file
- si poteva gestire il token del bot mediante un file apposito di configurazione da mettere in .gitignore senza dover modificare il codice per cambiare token
Non male come progetto 👍
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
No labels