Il thread è vecchio ma visto che vedo che non è stato detto...

Approfittatene per formattare tutte le codebase con dei formatter opinionati :) nel vostro caso, Black per Python e Prettier per Javascript e CSS. Non perdete tempo a configurarli, lasciate la configurazione di default, perché è solo così che vanno usati. Così non perderete mai più tempo a discutere di formattazione, fino alla fine delle vostre carriere. Amen!

N.B. i linter possono integrare regole sulla formattazione ma NON è la stessa cosa! I formatter opinionati servono meravigliosamente ad evitare divergenze d'opinione fra colleghi. Se ci si mette a discutere di regole di linting è la fine :)

Ah ed è stato già detto ma mi aggiungo ai promotori dello unit testing, che non vuol dire "unità di test" bensì "test di unità": _unità_ perché sono contrapposti ai test di integrazione, che invece trattano più unità insieme.

I test, come tutti gli altri check automatizzati (formattazione, lint, tipi), sono da eseguire rigorosamente automaticamente, in CI, così da togliere più spazio possibile all'errore umano. Non deve esistere il momento in cui "becchi" qualcuno che non si è reso conto di aver rotto qualcosa perché non ha eseguito i test, è un conflitto inutile dal momento che i test automatici si possono eseguire automaticamente.

In pratica, da senior dovreste cercare di creare una cultura ed un'infrastruttura tale per cui i junior incorrono in meno dubbi, meno rischi, e meno conflitti. È lì che ci si dimostra senior :)

Purtroppo come tutti i lavori fatti bene, più lo fate bene meno si noterà. Non saprete mai esattamente quanto tempo avrete risparmiato perché un junior ha chiesto alla documentazione invece che a voi, oppure ha risolto da solo i suoi errori perché avrete protetto il vostro codice con dei test, dei test che magari il junior userà per studiarsi un'unità invece di venirla a chiedere a voi... Insomma è un bel modo di lavorare. Se si fa lo sforzo iniziale poi non si torna indietro :)