Tri des tickets

Django uses Trac for managing the work on the code base. Trac is a community-tended garden of the bugs people have found and the features people would like to see added. As in any garden, sometimes there are weeds to be pulled and sometimes there are flowers and vegetables that need picking. We need your help to sort out one from the other, and in the end, we all benefit together.

Like all gardens, we can aspire to perfection, but in reality there’s no such thing. Even in the most pristine garden there are still snails and insects. In a community garden there are also helpful people who – with the best of intentions – fertilize the weeds and poison the roses. It’s the job of the community as a whole to self-manage, keep the problems to a minimum, and educate those coming into the community so that they can become valuable contributing members.

De même, bien que nous visions que Trac soit une représentation parfaite de l’état de progression de Django, nous reconnaissons que cela n’arrivera pas. En distribuant la charge de maintenance de Trac à la communauté, nous acceptons qu’il y aura des erreurs. Trac est « majoritairement à jour » et nous assumons que dans certains cas il se trompe. Cela nous va. Nous sommes des perfectionnistes avec des délais !

Nous comptons sur la communauté pour qu’elle continue de participer, de garder les tickets dans un état le plus juste possible et qu’elle amène les sujets de discussion dans nos listes de diffusion lorsqu’il y a de la confusion ou des désaccords.

Django est un projet communautaire et chaque contribution est utile. Nous ne pouvons pas le faire sans vous !

Procédures de tri des tickets

Malheureusement, un certain nombre de signalement de bogues ou de demandes de fonctionnalités dans le système des tickets ne contiennent pas tous les détails requis. Certains tickets ont des correctifs, mais ceux-ci ne satisfont pas toujours à toutes les exigences d’un bon correctif.

Une manière d’aider est de « trier » des tickets que d’autres utilisateurs ont créés.

La plupart du flux de travail est basé autour du concept d’:ref:étapes de tri <triage-stages>` d’un ticket. Chaque étape définit le stade dans lequel se trouve un ticket dans sa durée de vie. En compagnie de quelques autres options à cocher, cet attribut indique de qui et de quoi dépend la suite de l’avancement du ticket.

Comme une image vaut mieux que mille mots, commençons par ceci :

Il existe deux rôles dans ce diagramme :

• Les fusionneurs : des personnes ayant accès de commit et qui sont responsables de rendre la décision finale pour la fusion des correctifs.
• Les trieurs de tickets : n’importe quel membre de la communauté Django choisissant de s’impliquer dans le processus de développement de Django. Notre installation Trac est volontairement ouverte au public et quiconque peut trier des tickets. Django est un projet communautaire et nous encourageons le tri par la communauté.

En guise d’exemple, voici le cycle de vie d’un ticket moyen :

• Alice crée un ticket et y associe une requête de contribution incomplète (pas de tests, implémentation non correcte).
• Bob révise la requête de contribution, marque le ticket avec les coches « Accepted », « needs tests » et « patch needs improvement », puis ajoute un commentaire indiquant à Alice comme le correctif peut être amélioré.
• Alice met à jour la requête de contribution, ajoutant des tests (sans modifier l’implémentation). Elle décoche les deux cases correspondantes.
• Charlie révise la requête de contribution et enlève la coche « patch needs improvement » et ajoute son commentaire au sujet de l’amélioration de l’implémentation.
• Alice met à jour la demande de contribution, corrigeant l’implémentation. Elle enlève le drapeau « patch needs improvement » (correctif nécessite amélioration).
• Daisy relit la demande de contribution et marque le ticket comme « Ready for checkin » (prêt pour l’intégration).
• Jacob, un fusionneur, révise la demande de contribution et l’intègre dans le code principal (« merge »).

Certains tickets n’ont pas besoin d’autant d’interventions, mais il y a aussi des tickets qui en nécessitent beaucoup plus.

Étapes de tri

Nous décrivons ci-dessous plus en détails les diverses étapes par lesquelles un ticket passe durant son parcours de vie.

Non traité (Unreviewed)

Le ticket n’a pas été relu par quelqu’un qui se sentait qualifié pour juger si ce ticket contient un problème réel, une fonctionnalité raisonnable ou s’il devrait plutôt être fermé pour une ou différentes raisons.

Accepté (Accepted)

La grande zone grise ! La signification absolue de « accepté » est que le problème décrit dans le ticket est valide et se trouve dans un état où on peut travailler sur sa résolution. Au-delà de cela, d’autres aspects sont à prendre en considération :

• Accepted, sans autre option

  Le ticket est valable, mais personne n’a encore soumis de correctif. Cela signifie souvent que vous pouvez sans autre commencer à écrire un correctif correspondant. C’est généralement plus vrai dans les cas de bogues acceptés que pour les fonctionnalités acceptées. Un ticket pour un bogue qui a été accepté signifie que le problème a été vérifié par un moins un trieur comme un bogue légitime, et devrait donc être corrigé si possible. Une nouvelle fonctionnalité acceptée pourrait ne signifier qu’un trieur a pensé que la fonctionnalité serait intéressante à avoir, mais ceci ne représente pas forcément un consensus général ni n’implique avec certitude qu’un correctif sera accepté pour cette fonctionnalité. Recherchez plus d’avis avant d’écrire un correctif conséquent si vous êtes dans le doute.

• Accepted + Has Patch

  Le ticket est en attente de personnes qui vont examiner le correctif proposé. Cela implique de télécharger le correctif et l’essayer, en vérifiant qu’il contienne des tests et de la documentation, en lançant la suite de tests après l’application du correctif, et en indiquant son avis sur le ticket.

• Accepted + Has Patch + Needs …

  Cela signifie que le ticket a été examiné, mais qu’il a été jugé nécessaire de le retravailler quelque peu. « Needs tests » et « Needs documentation » s’expliquent par eux-mêmes. « Patch needs improvement » est généralement accompagné d’un commentaire sur le ticket expliquant ce qui est nécessaire pour améliorer le code.

Prêt pour le commit (Ready For Checkin)

Le ticket a été relu et examiné par tout membre de la communauté autre que la personne qui a fourni le correctif et l’examen n’a pas révélé de manquement qui empêcherait le correctif d’être prêt au commit. Un fusionneur devra ensuite donner un dernier coup d’œil au correctif avant de procéder au commit.

Il y a beaucoup de requêtes de contribution. L’examen de votre correctif peut prendre du temps. Consultez la FAQ de contribution au code pour d’autres questions à ce sujet.

Un jour peut-être (Someday/Maybe)

This stage isn’t shown on the diagram. It’s used sparingly to keep track of high-level ideas or long-term feature requests.

Ces tickets sont plus rares et de manière générale moins utiles car ils ne ciblent pas des problèmes concrets. Ce sont des demandes d’amélioration que nous considérons pouvoir ajouter à Django dans le futur si un excellent correctif est envoyé. Leur priorité est faible.

Autres attributs de tri

Un certain nombre de drapeaux, apparaissant sous forme de cases à cocher dans Trac, peuvent être définis pour un ticket :

Has patch (possède un correctif)

Cela signifie que le ticket possède un correctif associé. Ceux-ci sont analysés pour estimer si le correctif est « bon ».

Les trois champs suivants (Needs documentation, Needs tests, Patch needs improvement) ne s’appliquent que lorsqu’un correctif est disponible.

Needs documentation (a besoin de documentation)

Ce drapeau est utilisé pour les tickets ayant un correctif, mais pas encore de documentation. La documentation complète des fonctionnalités est un prérequis avant de pouvoir faire entrer du code dans Django.

Needs tests (a besoin de tests)

Ce drapeau indique que le correctif a besoin de tests unitaires associés. Encore une fois, il s’agit d’un élément obligatoire pour un correctif valable.

Patch needs improvement (le correctif a besoin d’améliorations)

Ce drapeau signifie que même si le ticket possède un correctif, il n’est pas encore prêt à être commité. Cela peut signifier que le correctif n’est plus applicable, que son implémentation est défaillante ou que le code ne correspond pas aux standards exigés.

Easy pickings (simple à corriger)

Désigne les tickets qui nécessitent de petits correctifs assez simples.

Type

Les billets doivent être classés par type parmi :

• New Feature (nouvelle fonctionnalité)

  Pour ajouter quelque chose de nouveau.

• Bug

  Quand une chose existante est cassé ou n’a pas le fonctionnement attendu.

• Nettoyage/optimisation

  Lorsque rien n’est cassé mais quelque chose pourrait être plus propre, mieux, plus rapide, plus puissant.

Component (composant)

Les tickets doivent être classés en composants indiquant quelle partie du code de Django ils concernent. Cela aide à organiser les tickets et à les rendre plus facilement retrouvables.

Severity (sévérité)

The severity attribute is used to identify blockers, that is, issues that should get fixed before releasing the next version of Django. Typically those issues are bugs causing regressions from earlier versions or potentially causing severe data losses. This attribute is quite rarely used and the vast majority of tickets have a severity of « Normal ».

Version

Il est possible d’utiliser l’attribut version pour indiquer la version dans laquelle le bogue signalé a été identifié.

UI/UX

Ce drapeau est utilisé pour les tickets qui sont liés aux questions d’interface utilisateur et d’expérience d’utilisation. Par exemple, ce drapeau serait approprié pour des fonctionnalités de présentation à l’utilisateur dans les formulaires ou l’interface d’administration.

Cc

Il est possible d’ajouter son nom d’utilisateur ou son adresse électronique dans ce champ pour être averti lorsque de nouvelles contributions sont apportées au ticket concerné.

Keywords (mots-clés)

With this field you may label a ticket with multiple keywords. This can be useful, for example, to group several tickets on the same theme. Keywords can either be comma or space separated. Keyword search finds the keyword string anywhere in the keywords. For example, clicking on a ticket with the keyword « form » will yield similar tickets tagged with keywords containing strings such as « formset », « modelformset », and « ManagementForm ».

Fermeture de tickets

Lorsqu’un ticket a terminé son cycle de vie utile, il est temps de le fermer. Cependant, la fermeture d’un ticket est une grande responsabilité. Il faut être certain que le problème est réellement réglé, et il faut garder à l’esprit que celui qui a ouvert le ticket pourrait ne pas être heureux de voir son ticket fermé (sauf si une correction a été appliquée !). Si vous n’êtes pas sûr qu’un ticket doit être fermé, laissez un commentaire avec votre raisonnement.

Si vous fermez réellement un ticket, vous devez toujours vous assurer des éléments suivants :

• Être certain que le problème est résolu.
• Laisser un commentaire expliquant la décision de fermer le ticket.
• S’il existe une manière d’améliorer le ticket pour qu’il soit réouvert, le faire savoir.
• Si le ticket est un doublon, faire référence au ticket original. De même, ajouter une référence croisée en ajoutant un commentaire sur le ticket original, ce qui permet d’avoir accès à davantage d’informations sur le bogue signalé ou la fonctionnalité demandée.
• Soyez poli. Personne n’aime voir son ticket être fermé. Cela peut être frustrant et même décourageant. La meilleure manière d’éviter de détourner les gens de la contribution à Django est de rester poli et amical, et d’offrir des suggestions sur la manière d’améliorer le ticket ainsi que d’autres tickets à venir.

Un ticket peut être résolu de plusieurs façons :

• fixed (résolu)

  Utilisé quand un correctif a été intégré à Django et que le problème est réglé.

• invalid (non valide)

  Utilisé lorsque le ticket est jugé incorrect. Cela signifie que le problème du ticket est en fait le résultat d’une erreur de l’utilisateur ou que le problème décrit s’applique à un autre composant que Django, ou encore qu’il ne s’agit ni d’un signalement d’erreur, ni d’une demande d’amélioration (par exemple, certains utilisateurs débutants ouvrent des tickets pour des demandes de support).

• wontfix (ne sera pas résolu)

  Used when someone decides that the request isn’t appropriate for consideration in Django. Sometimes a ticket is closed as « wontfix » with a request for the reporter to start a discussion on the django-developers mailing list if they feel differently from the rationale provided by the person who closed the ticket. Other times, a mailing list discussion precedes the decision to close a ticket. Always use the mailing list to get a consensus before reopening tickets closed as « wontfix ».

• duplicate (doublon)

  Utilisé lorsqu’un autre ticket recouvre le même problème. En fermant les tickets doublons, la discussion reste centralisée à un seul endroit, ce qui aide tout le monde.

• worksforme (marche pour moi)

  Utilisé lorsque le ticket ne contient pas assez de détails pour reproduire l’anomalie originale.

• needsinfo (besoin d’informations)

  Utilisé lorsque le ticket ne contient pas assez d’informations pour reproduire le problème signalé, mais qu’il est potentiellement valable. Le ticket devrait être réouvert après que des informations supplémentaires ont été fournies.

Si vous pensez que le ticket a été fermé par erreur, soit que le problème demeure, soit qu’il est réapparu ailleurs ou encore que les trieurs se sont trompés, vous pouvez réouvrir le ticket en fournissant des informations complémentaires. Encore une fois, évitez de réouvrir des tickets qui ont été fermés comme « wontfix », mais dans ce cas, préférez la discussion du problème sur la liste django-developers.

Commet puis-je aider à trier les tickets ?

Le processus de tri est principalement mené par des membres de la communauté. Vraiment, TOUT LE MONDE peut aider.

Pour vous impliquer, commencez par créer un compte sur Trac. Si vous disposez d’un compte mais que vous avez oublié le mot de passe, vous pouvez le réinitialiser en utilisant la page de réinitialisation de mot de passe.

Puis, vous pouvez aider en :

• Fermant les nouveaux tickets (« Unreviewed ») comme « invalid », « worksforme », « duplicate » ou « wontfix ».
• Fermant les nouveaux tickets (« Unreviewed ») comme « needsinfo » lorsque la description est trop vague pour qu’on puisse imaginer une résolution ou lorsqu’il s’agit de demandes de fonctionnalité qui nécessitent une discussion sur la liste django-developers.
• Corrigeant les coches « Needs tests », « Needs documentation » ou « Has patch » pour les tickets où elles ne sont pas correctement définies.
• Activant la coche « Easy pickings » pour les tickets de faible portée et relativement évidents.
• Définissant le type des tickets qui ne sont pas encore catégorisés.
• Vérifiant que les anciens tickets sont toujours valables. SI un ticket n’a pas vu d’activité depuis longtemps, il est possible que le problème ait été résolu dans l’intervalle sans que le ticket ait été fermé.
• Identifiant des tendances et des thèmes dans les tickets. Si de nombreux signalements de bogues concernent une partie particulière de Django, cela peut indiquer que cette partie de code a besoin de réécriture. Si une tendance se dégage, il est suggéré d’amener une discussion (en faisant référence aux tickets concernés) sur la liste django-developers.
• Vérifiant si les correctifs soumis par d’autres utilisateurs sont corrects. S’ils sont corrects et qu’ils contiennent la documentation et les tests nécessaires, vous pouvez faire passer le ticket à l’état « Ready for Checkin ». Dans le cas contraire, écrivez un commentaire expliquant pourquoi et cochez les cases correspondantes (« Patch needs improvement », « Needs tests », etc.).

Note

La page des rapports contient des liens vers de nombreuses requêtes Trac utiles, y compris certaines orientées sur le triage des tickets et la relecture de correctifs comme suggéré ci-dessus.

Vous pouvez aussi trouver plus de Conseils pour les nouveaux contributeurs.

Cependant, nous demandons à tous les membres de la communauté de respecter les principes suivants en travaillant sur la base de données des tickets :

• Please don’t promote your own tickets to « Ready for checkin ». You may mark other people’s tickets that you’ve reviewed as « Ready for checkin », but you should get at minimum one other community member to review a patch that you submit.
• S’il vous plaît, ne revenez pas sur une décision dans d’abord écrire un message à django-developers pour trouver un consensus.
• Si vous n’êtes pas certain de devoir faire une modification, ne la faites pas, mais laissez plutôt un commentaire avec vos préoccupations sur le ticket, ou écrivez un message sur django-developers. Il est légitime de ne pas savoir, mais votre opinion compte tout de même.

Recherche d’origine d’une régression avec bisect

Une régression est un bogue présent dans une version plus récente de Django et qui ne l’était pas avant. Un élément d’information extrêmement utile est le commit qui a introduit la régression. Le fait de connaître le commit qui est la cause du changement de comportement aide à découvrir si la modification était intentionnelle ou s’il s’agit d’un effet de bord inattendu. Voici comment vous pouvez trouver ce commit.

Commencez par écrire un test de régression du problème pour la suite de tests de Django. Par exemple, nous supposons que nous recherchons une régression dans les migrations. Après avoir écrit le test et confirmé que celui-ci échoue avec le code le plus récent (branche main), placez-le dans un fichier séparé que vous pouvez exécuter de manière indépendante. Pour notre exemple, nous supposons que nous avons placé le test dans le fichier tests/migrations/test_regression.py, qui peut être exécuté avec

$ ./runtests.py migrations.test_regression

Puis, nous marquons le point actuel dans l’historique comme « mauvais » (bad) dans la mesure où le test échoue

$ git bisect bad
You need to start by "git bisect start"
Do you want me to do it for you [Y/n]? y

Now, we need to find a point in git history before the regression was introduced (i.e. a point where the test passes). Use something like git checkout HEAD~100 to check out an earlier revision (100 commits earlier, in this case). Check if the test fails. If so, mark that point as « bad » (git bisect bad), then check out an earlier revision and recheck. Once you find a revision where your test passes, mark it as « good »:

$ git bisect good
Bisecting: X revisions left to test after this (roughly Y steps)
...

Nous sommes maintenant prêt pour la partie sympa : utilisez git bisect run pour automatiser la suite du processus

$ git bisect run tests/runtests.py migrations.test_regression

Vous devriez voir git bisect utiliser une recherche binaire pour extraire automatiquement les révisions entre les bons et mauvais commits jusqu’à ce qu’il trouve le premier commit « mauvais » où le test échoue.

À ce stade, reportez vos résultats sur le ticket Trac et ajoutez le test de régression en pièce jointe. Lorsque quelqu’un écrira une correctif pour ce problème, il disposera déjà de votre test comme point de départ.