Review

[Helias]

• vi siete ammazzati di lavoro 🚀 (non era richiesto)
• ho visto che avete lavorato tutti in maniera abbastanza bilanciata, deduco che il vostro livello è alto e siete tutti e 3 abbastanza preparati 👍
• ho revisionato gli unit tests, mi sembrano ok 🟢
• avete scritto abbastanza PRs ed utilizzato propriamente i conventional commits
• avete tipizzato sufficientemente il codice

Il progetto per me è completo, se volete continuare a lavorarci ben venga, comunque penso che siete il team che ha lavorato di più al proprio progetto 🚀 🥇 . ⚠️ Perfavore, scrivetemi tutti i nomi di chi sta partecipando, purtroppo non mi sono segnato ai tempi quali studenti erano associati a questo progetto (ho individuato solo Bruno Montalto dal nickname), inoltre non ho ben tenuto traccia del gruppo perchè il progetto non è stato presentato qui https://github.com/UNICT-Quality-Development/qd-projects-2022/issues

(appena lo farete, provvederò ad inviare agli attestati di QD)

In ogni review ho sempre lasciato alcuni suggerimenti opzionali nel momento in cui vorreste continuare a migliorare il progetto (a tempo perso):

• directory structure, ci sono un sacco di file messi nella cartella root del progetto, andrebbe creata una directory structure migliore
• Table.sql.bak non so se è normale che stia lì, nei sorgenti non vanno messi backup (stessa cosa per queries.sql.bak)
• oltre a requirements.txt potreste creare requirements_dev.txt
• fixate la pipeline aggiungendo pylint (https://github.com/UNICT-Quality-Development/qd-starter-pack/pull/25), comunque la pipeline va aggiunta all'inizio non alla fine, perchè serve come supporto durante lo sviluppo
• fixate tutto ciò che vi suggerisce pylint, migliorerete la qualità del codice
• anche la key self.pem non penso debba essere presente in una repository forse avrebbe più senso definire come generarla nel readme
• migliorate il readme

Avete fatto veramente un sacco, spero vi siate divertiti 🥳

P.S. = so che non avete "consegnato il progetto via email" ma avevo messo il "watch" nella vostra repository ed avevo notato qualche notifica, inoltre stavo revisionando tutti i progetti di QD che mi sono arrivati fin'ora non voglio che vi ammazzate per la scadenza di mezzanotte perchè avete già fatto troppo

[TheRealGioviok]

Grazie per il feedback! Procedo a mandare i nomi via email. Un paio delle cose che hai citato le stavamo già aggiungendo, mentre altre, come i file committati, sono li da vecchi commit dove qualcuno (vero @BrunoMontalto ?) aveva dimenticato di aggiungerli al gitignore. Grazie ancora per tutto!