Il y a des nouveaux "commiters" depuis quelques semaines : Julien, Dominique et Bastien sont venus rejoindre l'équipe de Jelix :-). Cela va soulager un peu mon travail de "commiter". J'ai aussi commencé à affecter des parties du code de Jelix à certains commiters, des parties qu'ils connaissent bien, bien sûr. Cela va soulager mon travail de "reviewer".

En effet, ces derniers semaines, j'ai plus passé mon temps à faire de la review de code et du commit que de coder des nouveaux trucs. Donc maintenant, je délègue, vu qu'il y a des contributeurs réguliers. Pour rappel, dans le projet Jelix, nous appliquons le principe de peer-review. Chaque modification dans le code est d'abord revue par un autre développeur avant d'être inclus dans le dépôt subversion. Cela garantie un code d'une certaine qualité, cela permet de corriger les bugs en amont, et d'avoir une certaine stabilité du trunk. Seule exception : ce que je développe n'est pas encore revue, car bien souvent il arrive que je fasse des trucs sur des parties de code que personne ne maîtrise encore (pas par faute d'incompétence de la part des autres développeurs, mais tout simplement parce qu'ils n'ont pas encore eu le temps de se plonger dans le coeur de Jelix). Mais j'espère bien pouvoir faire "relire" mon code :-)

Le peer-review est bénéfique pour le développeur et pour le reviewer. Pour le développeur car cela lui permet de faire du code de meilleur qualité si il n'est pas très expérimenté, ou si il est nouveau dans le projet et qu'il n'a pas forcément conscience des conséquences de telle ou telle chose dans son code. Pour le reviewer aussi : cela lui permet de mieux suivre les modifications qui se font dans les composants dont il a la charge. Il est le garant aussi de la cohérence du code, des fonctionnalités. Il peut aussi avoir l'occasion d'apprendre des choses, ou encore que le patch lui donne des idées pour améliorer encore plus. Bien sûr, les patchs d'un reviewer sont aussi revu par d'autres reviewer. On se corrige donc mutuellement. Cela augmente la qualité générale du code.

Par exemple, Sylvain prépare un superbe patch sur le support de SOAP dans Jelix. Il connait bien SOAP mais pas moi, et je connais bien Jelix (si si :-)) mais lui ne maitrise pas forcément toutes les arcanes du coeur de Jelix. Ainsi j'ai pu le guider dans la manière d'intégrer le support SOAP dans Jelix, et son patch m'a permis (obligé en fait) d'apprendre l'API PHP de SOAP, et d'avoir des réponses auprès de Sylvain sur des questions que je me posais sur le fonctionnement de SOAP (je n'ai jamais fait de SOAP...). Pour info, le support SOAP sera donc disponible dans Jelix 1.1.

À part ça, Bastien travaille activement sur une meilleur page "d'accueil" quand on crée une nouvelle application (Ce sera mieux qu'une page blanche :-). Julien corrige des bugs un peu partout et améliore jLocale. Et moi j'ai travaillé sur le module jCommunity, et j'ai commencé le développement d'un système d'installation automatique pour les modules et les applications. Après je m'attaquerais sérieusement aux améliorations prévus dans jForms, le système de formulaire de Jelix qui plait beaucoup apparement :-).

J'ai décidé de faire une news "plus officielle", chaque semaine sur jelix.org, listant les dernières avancées sur le projet, m'inspirant des "commit-digest" de KDE (et d'autres projets). J'ai commencé hier...