Skip to content

Instantly share code, notes, and snippets.

@gunzip

gunzip/code-review.md

Last active June 9, 2021 14:20
Show Gist options
  • Save gunzip/f262bd15b739e100f685bfc9bd1840b9 to your computer and use it in GitHub Desktop.
Save gunzip/f262bd15b739e100f685bfc9bd1840b9 to your computer and use it in GitHub Desktop.
Download ZIP
Raw
code-review.md

Comunicazione & progettazione

  • limitare l'utilizzo della comunicazione sincrona (slack) per q&a
  • partire dai doc gdrive per l'analisi dei requisiti, inserire lì i commenti e taggare le persone che possono rispondere
  • durante la standup howdy alla domanda 'hai bisogno di aiuto' taggare le stesse persone per sollecitare una risposta
  • usare le storie su pivotal per discutere l'implementazione quando i requisiti sono chiari
  • usare le PR per discutere i dettagli implementativi
  • inserire eventualmente dei commenti inline nelle PR per spiegare alcune scelte progettuali (non dovrebbe essere necessario se si usano bene i commenti nel codice :-)

Pattern

Repository con funzionalità condivise

Backend (express)

Functions

Materiale di studio

fp-ts / io-ts

Programming

Azure functions

Focus sulla manutenibilità del codice

naming esplicito e comunicativo

there are two hard problems in computer science cache invalidation and naming things

evitare codice non necessario e "alias"

best code is no code

evitare di "estrarre" i valori dai tipi fp-ts

use map / chain / fold instead of isLeft / isRight / .value

scrivere commenti utili e esplicativi

ritornare taskeithers al posto di promise

usare taskify per convertire i metodi che usano callback*

meglio concatenare i task piuttosto che gestire imperativamente costrutti condizionali

https://grossbart.github.io/fp-ts-recipes/#/async?a=work-with-a-list-of-tasks-in-parallel

preferire i tipi condivisi (io-ts-commons) a "string"

evitare di committare codice non raggiungibile o non utilizzato

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment