developer.jelix.org is not used any more and exists only for history. Post new tickets on the Github account.
developer.jelix.org n'est plus utilisée, et existe uniquement pour son historique. Postez les nouveaux tickets sur le compte github.

Changes between Version 20 and Version 21 of fr/patchs


Ignore:
Timestamp:
Jan 1, 2012, 10:03:36 PM (8 years ago)
Author:
laurentj
Comment:

--

Legend:

Unmodified
Added
Removed
Modified
  • fr/patchs

    v20 v21  
    3333Il se peut que ce soit un patch incomplet (vous n'avez pas le temps de le finir, vous voulez passer la main, ou alors ce n'est qu'une première étape etc...), dans ce cas là, vous vous arrêtez là. (Toutefois un commentaire ne sera pas de trop pour indiquer le pourquoi, où vous en êtes etc..)
    3434
    35 Sinon, si votre patch vous semble prêt, sélectionnez "review?" dans le champs review du ticket. Cela veut dire que vous demandez une review.
     35Sinon, si votre patch vous semble prêt, choisissez "ask a review" dans le ticket. Cela veut dire que vous demandez une review. Un "reviewer" regardera alors votre patch, et vous dira en retour si il y a des améliorations à faire ou si c'est bon. Si ce n'est pas bon, il faudra faire les améliorations demandées et proposer un nouveau patch que vous attacherez sur le même ticket. Et vous demanderez alors une nouvelle review. Et ainsi de suite.
    3636
    37 Un "reviewer" regardera alors votre patch, et vous dira en retour si il y a des améliorations à faire (il mettra "review-") ou si c'est bon ("review+").
    38 
    39 Si il y a "review-", il faudra faire les améliorations demandées et proposer un nouveau patch que vous attacherez sur le même ticket. Et vous demanderez alors une nouvelle review ("review?"). Et ainsi de suite.
    40 
    41 Une fois que vous avez "review+", si vous avez les droits pour commiter dans le dépôt mercurial, vous incluez alors vos modifications dans le dépôt. Sinon un "commiter" le fera à votre place. Voir [wiki:fr/subversion les conventions pour commiter].
     37Une fois que le patch est accepté, si vous avez les droits pour commiter dans le dépôt git, vous incluez alors vos modifications dans le dépôt. Sinon un "commiter" le fera à votre place. Voir [wiki:fr/commits les conventions pour commiter].
    4238
    4339
    4440=== Vous êtes un contributeur actif ===
    4541
    46 Si vous fournissez des patches régulièrement (indépendants ou non), la meilleure façon est de "forker" le depot de jelix dans votre compte github.
     42Si vous fournissez des patchs régulièrement (indépendants ou non), la meilleure façon est de "forker" le dépôt de jelix dans votre compte github.
    4743
    4844Créez une branche dans votre "fork", pour chaque modification distincte (pour chaque ticket en gros). Cela permettra alors de fusionner les branches qui ont été acceptées sans inclure vos modifications non terminées ou refusées.
    4945
    50 Pour les petites modifications, essayez de limiter le nombre de commit. Cela sera plus facile de faire la review, et cela permettra de garder un historique clair. Pour les modifications plus conséquentes, il est préférable bien sûr d'avoir plusieurs commits, un pour chaque étape. Afin d'obtenir un historique compréhensible, essayer de faire des commits clairs. Chaque changement devrait correspondre à une étape spécifique de modifications.
     46Pour les petites modifications, essayez de limiter le nombre de commit. La review sera plus simple et cela permettra de garder un historique clair. Pour les modifications plus conséquentes, il est préférable bien sûr d'avoir plusieurs commits, un pour chaque étape. Afin d'obtenir un historique compréhensible, essayer de faire des commits clairs. Chaque changement devrait correspondre à une étape spécifique de modifications.
    5147
    5248Lisez [wiki:fr/commits la page sur les commits] pour le format des messages des commits.
    5349
    5450
    55 Quand votre modification est prête, faites un "pull request" sur github. Si il y a un ticket correspondant sur developer.jelix.org, vous pouvez aussi informer sur ce ticket vos travaux. Un reviewer vérifiera alors votre code, proposera peut-être des améliorations à faire, et ensuite importera vos changements dans le dépôt officiel.
     51Quand votre modification est prête, faites un "pull request" sur github. Si il y a un ticket correspondant sur developer.jelix.org, vous pouvez aussi informer sur ce ticket vos travaux (en choisissant "ask a review" dans le ticket par exemple). Un reviewer vérifiera alors votre code, proposera peut-être des améliorations à faire, et ensuite importera vos changements dans le dépôt officiel.
    5652
    5753