DenyR96 / Sanremo-ranking

E' un programma in python che permette di: visualizzare la classifica di Sanremo in uno specifico anno e visualizzare link di youtube con/senza testo della canzone di ogni classifica
1 stars 1 forks source link

Quick review #7

Open Helias opened 2 years ago

Helias commented 2 years ago

Vedo che il progetto sta prendendo forma, se per caso vi sta rubando troppo tempo vi consiglio di ridurre le funzionalità, scrivere gli unit test e basta, il progetto deve essere molto semplice e serve ad imparare bene git, github e un po' anche python ed unit test.

Una cosa che dovreste fare è quella di seguire attentamente le linee guida in particolare:

Noto che lavorate al progetto dal 5 Giugno 👍 , se per caso pensate che vi stia tenendo troppo tempo, ridimensionate le funzionalità da apportare e scrivete qualche unit test.

Helias commented 2 years ago

L'ho provato, già mi sembra ci siano tutte le funzionalità previste, anche quelle bonus 💪 , mancano solo gli unit test, vedete cosa riuscite a fare al livello di unit test e code coverage.

Per il resto mi sembra ok 💪

Fate qualche unit-test, cercate di raggiungere il 75% di code coverage testando le varie classi, evitate di testare il Main perchè potreste riscontrare problemi ( https://t.me/c/1787645455/120 ).

image
DenyR96 commented 2 years ago

Grazie mille per i feedback e l'aiuto che ci stai dando. Infatti la cosa più importante che ci rimane sono proprio gli unit test a cui lavoreremo in questi giorni.

DenyR96 commented 2 years ago

Aggiungeremo presto anche la parte sulla pipeline