Provando a far partire i tests in realtà il test test_ASFDPrinter falliva, perchè mse di assert mse == 0 dava 0.00..1 == 0, convertendolo in int() funziona tutto (ho provato con python3.9.10, lo stesso della pipeline, eppure nel mio PC mi ha dato questo errore).
Feedback del progetto:
- conventional commits ✅
- avete scritto un buon README.md ✅
- raggiunta code coverage 100% ✅ 🚀
- il codice è abbastanza tipizzato ✅
- il codice ha molti commenti ✅
- avete creato diverse classi ed utilizzato i tipi generici con Stack e Queue ✅ 💯
- bellissimo il file test.sh, non so perchè ma localmente non mi funziona però è una buona pratica eseguire i test prima di effettuare il push, non a caso esiste "husky" che fa questo, ovvero, al "git push" esegue automaticamente dei tests (o comandi bash) e blocca il push nel caso in cui i tests falliscono ✅ 🚀 🥇
- avete fatto solo 6 PR, sono un po' poche per il numero di membri e la grandezza del progetto ❌
- minor: alcuni nomi di variabili e funzioni non rispettano lo snake_case come
getSigma -> get_sigma
- potevate commentare di più le PR tra voi suggerendovi piccole migliorie come le regole dei nomi oppure es.
Union[int, None] -> Optional[int]
- minor: potevate aggiungere
if __name__ == "__main__": main() per poter avviare il file main mediante python (non sono riuscito ad avviarlo eccetto spostando src/main.py nella cartella superiore)
Il progetto sembra molto elaborato, bravi! 👍💪
Provando a far partire i tests in realtà il test
test_ASFDPrinterfalliva, perchèmsediassert mse == 0dava0.00..1 == 0, convertendolo inint()funziona tutto (ho provato con python3.9.10, lo stesso della pipeline, eppure nel mio PC mi ha dato questo errore).Feedback del progetto:
getSigma->get_sigmaUnion[int, None]->Optional[int]if __name__ == "__main__": main()per poter avviare il file main mediante python (non sono riuscito ad avviarlo eccetto spostando src/main.py nella cartella superiore)Il progetto sembra molto elaborato, bravi! 👍💪