REX
Périmètre 43

avec la tendance météo ☀️ 🌤 ☁️ 🌧 🌩

☀️ Revues de code

  • Agréablement surpris par la pertinence des commentaires alors que certains ne s'appliquent pas leur propres commentaires (!)
  • Vertueux
  • Diffusion de la connaissance
  • Lève des questions techniques
  • Uniformise les pratiques

🌤 Les PRs

  • Dynamique appréciée
  • Tout le monde joue le jeu
  • Système du binome satisfaisant
  • Augmente considérablement la qualité du code avant livraison sur branche dev-Xxx
  • Développeurs plus responsables et plus attentifs
  • Echanges constructifs
  • Workflow (défini dans le guide du dev) parfois non respecté
  • Attention à ne pas laisser trainer trop longtemps ses PRs (problème de merge, fraicheur du sujet)

☀️ Les démos

  • Peu de sujets / demos en 43
  • Responsabilise chacun
  • Pousse à communiquer et à tester 
  • Meilleure qualité supposée en réception test interne
  • Permet de lever des alertes

🌤 Questions / réponses techniques

  • Channel #dev-issues vivant
  • Peu de sujets de junior sur les réunions techniques hebdo
  • Conventions globalement bien appliquées dans le code

☁️ Test unitaire

  • Peu de prise d'initiative
  • TU trop souvent cassé
  • Mais, volonté et prise de conscience collective de son intérêt
  • AssertJ, Mockito, ... rendent l'écriture de TU plus "amusante" / plus haut niveau

🌧 Documentation technique (JavaDoc / TSDoc / Unison)

  • Manque d'initiative
  • Manque de retro documentation quand on creuse un sujet historique (méthodes ou variables mal nommées, mauvais découpage, ...)
  • Manque de nouvelle documentation
  • Efforts ou prise de conscience à prendre en 44 pour responsabiliser chacun à produire de la documentation (quand c'est nécessaire)
  • Peut être un problème d'expérience ? => Capacité à estimer un besoin de documentation

🌧 Rigueur

  • Git - Commit mauvaises branches
  • Git - Commit mal nommé
  • TU - cassé
  • TU - non joué
  • PR - mal nommé
  • PR - pas de reviewer assigné
  • Jira - worfklow non respecté
  • Jira - logs de temps restants / estimé

 

=> de nombreux progrès à faire sur tous ces secteurs

☀️ Passes Sentry

  • Beaucoup de progrès
  • Communication sur Slack dédié
  • Recherche des issues existants
  • Recoupage avec d'autres problèmes
  • Régularité respectée

🌤

REX 43

By Mathieu Castets