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
REX 43
- 79