Treball en equip

Nous aspectes

Treball en equip

1. Daily meetings

2. Pull Requests

3. Code Reviews

Daily Meetings

Daily Meetings

Quan? On?

• Cada dia, a la mateixa hora
• Curta duració 10-15 min (dempeus)
• Pissarra

Daily Meetings

Què aporten?

• Principi GIFTS
• Millora la comunicació a l'equip: autoconeixement
• Espai per a que cap d'equip comuniqui novetats
• En cas d'oblit, es demana aprovació de PRs

 'Good start, Improvement, Focus, Team, Status'

Daily meetings

Guió

1. ¿Què vam fer ahir?
    
2. ¿Què farem avui?
    
3. ¿Què ens està bloquejant / bloquejarà?

Daily meetings

Objectiu: Swarming

"Concentrar los esfuerzos de todos en la historia de mayor prioridad"

Una vez hecho, trabajar en paralelo todo lo que sea posible, pero siempre evitando el cambio de contexto.

Así se evitan situaciones en las que cada uno cuenta su rollo mientras el resto piensa en sus cosas, debido a que los temas no tienen relación entre sí.

        @jgarzas

Pull Requests

give teams a way to review code

El propi equip valida les seves PR ...

Pull requests ...

... as Code Reviews

• Convencions d'estil pactades únicament al principi, configurades al IDE/Editor
  • indentació
  • convencions de noms (ex: camelCase)
  • posicions de les claus ...
     
• Anem generant de zero una checklist d'aspectes a comprovar i preguntes a respondre. Document VIU!

• Aquest canvi augmenta el deute tècnic?
• És aquest codi llegible?
• Té comentaris?
• És coherent respecte a l'arquitectura pactada?
• Segueix les convencions d'estil?
• Aquest canvi causa que el test coverage decreixi?
• Aquest canvi causa que el rendiment decreixi?
• Aquest canvi causa que la seguretat decreixi?

Algunes preguntes de la checklist ...

Pull requests

Keep it simple, Keep it Short

Poques línies de codi  < 400

Beneficis: Creix la qualitat del codi

Pull requests

Beneficis: Creix el coneixement del codi

Pull requests

Les PR obliguen als membres de l'equip a educar als altres sobre bones pràctiques i convencions d'estil

Beneficis: Free mentorship

Pull requests

Beneficis: The BUS factor

El bus factor (hit-by-a-bus) consisteix en el número mínim de membres de l'equip que han de desaparèixer súbitament del projecte  abans de que aquest quedi estancat degut a la falta de personal amb el coneixement adient

Pull requests

Pull requests

• Actitud oberta: No és personal!
• Confiança en el criteri dels companys
• Voluntat de comentar el codi

Requereix de ...

Code Reviews

Code reviews

1. Integrades a les Pull Requests
2. Segons demanda
3. Periòdiques

Opcions ...

Code reviews

Periòdiques: Estratègies

• Quan la història s'acaba d'implementar, es fa review amb un company
• Dediquem 30 minuts conjuntament després de cada Sprint
• 1 vegada cada 4 sprints: 60 minuts d'una part random del codi per part dels desenvolupadors més experimentats

Opinions