Soumission de contributions

Nous sommes toujours reconnaissants pour toute contribution au code de Django. Il est vrai que les rapports de bogue avec contribution associée seront résolus bien plus rapidement que ceux sans solution proposée.

Corrections orthographiques et modifications de documentation

Si vous corrigez un problème vraiment trivial, par exemple en changeant un mot dans la documentation, la manière préférée de soumettre un correctif est de passer par les requêtes de contribution GitHub sans ouvrir un ticket Trac.

Consultez Travailler avec Git et GitHub pour plus de détails sur la manière d’utiliser les requêtes de contribution (« pull requests »).

« Appropriation » de tickets

Dans un projet de logiciel libre avec des centaines de contributeurs de par le monde, il est important de gérer efficacement la communication afin que le travail ne soit pas fait en double et que les contributions soient aussi efficaces que possible.

Ainsi donc, notre politique demande que les contributeurs « réclament » les tickets pour informer les autres développeurs qu’un bogue ou une fonctionnalité particulière est en cours de travail.

Si vous avez identifié une contribution que vous souhaiteriez apporter et que vous pensez en être capable (en fonction de vos capacités de codage, votre connaissance du fonctionnement de Django et vos disponibilités de temps), signalez-le en suivant ces étapes :

  • Connectez-vous avec votre compte GitHub ou créez un compte dans notre système de tickets. 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.

  • Si un ticket pour ce problème n’existe pas encore, créez-en un dans notre système de suivi des tickets.

  • Si un ticket pour ce problème existe déjà, assurez-vous que personne ne s’en occupe déjà. Pour cela, examinez la section Owned by du ticket. S’il est attribué à nobody, c’est qu’il est disponible. Sinon, quelqu’un d’autre est probablement en train de travailler sur ce ticket. Cherchez alors un autre bogue ou fonctionnalité ou contactez le développeur travaillant sur le ticket pour offrir votre aide. Si un ticket est attribué à quelqu’un depuis des semaines ou mois sans activité, il peut sans doute vous être attribué sans risque de conflit.

  • Log into your account, if you haven’t already, by clicking « GitHub Login » or « DjangoProject Login » in the upper left of the ticket page. Once logged in, you can then click the « Modify Ticket » button near the bottom of the page.

  • Claim the ticket by clicking the « assign to » radio button in the « Action » section. Your username will be filled in the text box by default.

  • Finally click the « Submit changes » button at the bottom to save.

Note

La fondation Django Software demande que toutes les personnes contribuant plus qu’un changement mineur dans Django signent et envoient un accord de licence de contribution. Ceci assure que la fondation Django Software possède des licences claires pour toutes les contributions permettant ainsi une licence claire pour tous les utilisateurs.

Responsabilité des « possesseurs » de tickets

Après vous être attribué un ticket, vous avez la responsabilité de travailler sur ce ticket dans un temps raisonnable. Si vous n’avez plus de temps à y consacrer, désattribuez le ticket… ou éviter de vous l’attribuer en premier lieu !

S’il n’y a aucun signe de progression sur un ticket attribué durant une ou deux semaines, un autre développeur peut demander que vous cédiez l’attribution afin qu’il ne soit plus monopolisé et que quelqu’un d’autre puisse s’y atteler.

Si vous vous êtes attribué un ticket et qu’il prenne beaucoup de temps à coder (des jours ou semaines), mettez tout le monde au courant en écrivant des commentaires sur le ticket. Si vous ne fournissez pas des mises à jour régulières et que vous ne répondez pas à une demande de rapport de progression, il est possible que le ticket soit attribué à quelqu’un d’autre.

Comme toujours, mieux vaut trop communiquer que pas assez !

Quels sont les tickets qui devraient être attribués ?

Passer par l’étape d’attribution de tickets est parfois surfait.

Dans le cas de petits changements tels que des corrections orthographiques de la documentation ou de petits bogues qui ne prennent que quelques minutes à corriger, vous n’avez pas besoin de passer par la gymnastique d’attribution des tickets. Soumettez directement vos modifications et voilà !

Il est toujours acceptable d’ajouter des liens vers des propositions dans un ticket si vous avez une proposition déjà prête, que le ticket soit attribué à quelqu’un ou pas.

Style de contribution

Assurez-vous que toute contribution remplit au minimum les exigences suivantes :

  • Le code exigé pour corriger un problème ou ajouter une fonctionnalité est une partie essentielle d’une solution, mais ce n’est pas la seule partie. Un bon correctif doit aussi inclure un test de régression pour valider le comportement ayant été corrigé et empêcher le problème de se reproduire à nouveau. De même, si d’autres tickets sont liés au code que vous avez écrit, mentionnez les numéros de tickets dans des commentaires de tests afin que l’on puisse aisément se référer aux discussions appropriées après que votre correctif aura été intégré et que les tickets auront été fermés.

  • Si le code ajoute une nouvelle fonctionnalité ou modifie le comportement d’une fonctionnalité existante, la modification doit aussi contenir de la documentation.

Lorsque vous pensez que votre travail est prêt à être relu, envoyez une requête de contribution GitHub. Si vous ne pouvez pas envoyer une requête de contribution pour une raison quelconque, vous pouvez également envoyer directement des correctifs dans Trac. SI vous faites cela, veuillez suivre ces indications.

  • Envoyez des correctifs dans le format renvoyé par la commande git diff.

  • Liez les correctifs à un ticket du système de suivi des tickets en utilisant le bouton « Attach file ». Évitez s’il-vous-plaît de placer un correctif dans la description ou dans un commentaire du ticket, sauf si le correctif contient une seule ligne.

  • Nommez le fichier du correctif avec une extension .diff ; cela permettra au système des tickets d’appliquer une coloration syntaxique correcte, ce qui est très utile.

Quelle que soit la manière de soumettre votre travail, respectez ces étapes.

  • Assurez-vous que votre code remplisse les exigences décrites dans notre liste de contrôle de contribution.

  • Cochez la case « Has patch » sur le ticket et vérifiez que les cases « Needs documentation », « Needs tests » et « Patch needs improvement » ne sont pas cochées. Cela fait que le ticket apparaît dans la file d’attente « Patches needing review » dans le tableau de bord de développement.

Contributions which require community feedback

A wider community discussion is required when a patch introduces new Django functionality and makes some sort of design decision. This is especially important if the approach involves a deprecation or introduces breaking changes.

The following are different approaches for gaining feedback from the community.

The Django Forum or django-developers mailing list

You can propose a change on the Django Forum or django-developers mailing list. You should explain the need for the change, go into details of the approach and discuss alternatives.

Please include a link to such discussions in your contributions.

Third party package

Django does not accept experimental features. All features must follow our deprecation policy. Hence, it can take months or years for Django to iterate on an API design.

If you need user feedback on a public interface, it is better to create a third-party package first. You can iterate on the public API much faster, while also validating the need for the feature.

Once this package becomes stable and there are clear benefits of incorporating aspects into Django core, starting a discussion on the Django Forum or django-developers mailing list would be the next step.

Django Enhancement Proposal (DEP)

Similar to Python’s PEPs, Django has Django Enhancement Proposals or DEPs. A DEP is a design document which provides information to the Django community, or describes a new feature or process for Django. They provide concise technical specifications of features, along with rationales. DEPs are also the primary mechanism for proposing and collecting community input on major new features.

Before considering writing a DEP, it is recommended to first open a discussion on the Django Forum or django-developers mailing list. This allows the community to provide feedback and helps refine the proposal. Once the DEP is ready the Steering Council votes on whether to accept it.

Some examples of DEPs that have been approved and fully implemented:

Obsolescence d’une fonctionnalité

Il existe plusieurs raisons pour que du code Django soit rendu obsolète :

  • Si une fonctionnalité a été améliorée ou modifiée de manière non rétrocompatible, l’ancienne fonctionnalité ou comportement sera rendu obsolète.

  • Il peut arriver que Django inclue un rétroportage d’une bibliothèque Python qui ne fait pas encore partie d’une version de Python prise en charge actuellement par Django. Lorsque Django n’a plus besoin de prendre en charge l’ancienne version de Python qui nécessitait d’inclure cette bibliothèque, l’inclusion de celle-ci sera rendue obsolète dans Django.

En accord avec la politique d’obsolescence, la première publication de Django qui rend obsolète une fonctionnalité (A.B`) doit produire un avertissement RemovedInDjangoXXWarning (où XX est la version de Django où la fonctionnalité sera supprimée) au moment où la fonctionnalité obsolète est appelée. Partant du principe que la couverture de tests est bonne, ces avertissements sont convertis en erreurs lorsque la suite de test est lancée <running-unit-tests> avec les avertissements activés : python -Wa runtests.py. Ainsi, quand vous ajoutez un avertissement RemovedInDjangoXXWarning, il est nécessaire d’éliminer ou de rendre silencieux tout avertissement produit lors de l’exécution des tests.

La première étape est d’enlever toute utilisation du comportement obsolète par Django lui-même. Ensuite, vous pouvez rendre silencieux les avertissements dans les tests qui testent la fonctionnalité obsolète en utilisant le décorateur ignore_warnings, soit au niveau d’un test isolé, soit sur une classe de tests :

  1. Pour un test particulier

    from django.test import ignore_warnings
from django.utils.deprecation import RemovedInDjangoXXWarning


@ignore_warnings(category=RemovedInDjangoXXWarning)
def test_foo(self): ...

  2. Pour tout un cas de test

    from django.test import ignore_warnings
from django.utils.deprecation import RemovedInDjangoXXWarning


@ignore_warnings(category=RemovedInDjangoXXWarning)
class MyDeprecatedTests(unittest.TestCase): ...

Vous devriez aussi ajouter un test pour l’avertissement d’obsolescence

from django.utils.deprecation import RemovedInDjangoXXWarning


def test_foo_deprecation_warning(self):
    msg = "Expected deprecation message"
    with self.assertWarnsMessage(RemovedInDjangoXXWarning, msg) as ctx:
        # invoke deprecated behavior
        ...
    self.assertEqual(ctx.filename, __file__)

Il est important d’inclure un commentaire RemovedInDjangoXXWarning au-dessus du code qui ne possède pas de référence vers un avertissement, mais qui devra être modifié ou supprimé à la fin de la période d’obsolescence. Cela pourrait concerner des points d’entrée qui auraient été ajoutés pour conserver le comportement précédent ou des éléments autonomes qui deviennent inutiles à la fin de la période d’obsolescence. Par exemple

import warnings
from django.utils.deprecation import RemovedInDjangoXXWarning


# RemovedInDjangoXXWarning.
def old_private_helper():
    # Helper function that is only used in foo().
    pass


def foo():
    warnings.warn(
        "foo() is deprecated.",
        category=RemovedInDjangoXXWarning,
        stacklevel=2,
    )
    old_private_helper()
    ...

Pour terminer, il reste à appliquer quelques mises à jour de la documentation Django :

  1. Si la fonctionnalité existante est documentée, marquez-la comme obsolète dans la documentation en utilisant l’annotation .. deprecated:: A.B. Ajoutez une courte description ainsi qu’une note sur la procédure de mise à jour le cas échéant.

  2. Ajoutez une description du comportement obsolète et la procédure de mise à jour le cas échéant aux notes de publication actuelles (docs/releases/A.B.txt) sous le chapitre « Features deprecated in A.B ».

  3. Ajoutez une ligne dans la planification d’obsolescence (docs/internals/deprecation.txt) sous la version adéquate en indiquant le code qui sera supprimé.

Après avoir accompli ces étapes, le procédé d’obsolescence est terminé. Dans chaque publication principale, tous les avertissements RemovedInDjangoXXWarning correspondant à la nouvelle version sont supprimés.

Contributions JavaScript

Pour des informations sur les contributions JavaScript, consultez la documentation Correctifs JavaScript.

Optimisation des correctifs

Les correctifs visant à produire une amélioration de performance doivent fournir des tests de performance démontrant l’impact avant et après le correctif et partager les commandes pour que les relecteurs puissent les reproduire.

Tests de performance django-asv

django-asv surveille la performance du code de Django au cours du temps. Ces tests peuvent être exécutés sur une requête de contribution en étiquetant la requête avec le mot benchmark. L’ajout de nouveaux tests de performance est hautement encouragé.

Liste de contrôle de contribution

Utilisez cette liste de contrôles pour relire une requête de contribution. Si vous relisez une requête de quelqu’un d’autre et qu’elle remplit tous les critères ci-dessous, veuillez définir la valeur « Triage Stage » du ticket Trac correspondant à « Ready for checkin ». Si vous avez écrit des commentaires d’amélioration sur la requête de contribution, veuillez cocher les options correspondantes du ticket Trac selon les résultats de votre relecture : « Patch needs improvement », « Needs documentation » et/ou « Needs tests ». Selon le temps et l’envie, les fusionneurs effectuent une relecture finale des tickets « Ready for checkin » et vont soit commiter les modifications, soit le renvoyer en statut accepté s’il manque encore certains éléments. Si vous espérez devenir fusionneur, le fait d’effectuer de sérieuses relectures de contributions est un bon moyen de gagner de la confiance.

En recherche d’un correctif à relire ? Consultez la section « Patches needing review » du tableau de bord de développement de Django.

Vous recherchez une relecture pour votre requête de contribution ? Assurez-vous que les drapeaux Trac sur le ticket correspondant soient bien définis afin que le ticket apparaisse dans cette file d’attente.

Documentation

  • Est-ce que la documentation peut être construite sans erreur (make html ou make.bat html sur Windows, à partir du répertoire docs) ?

  • Est-ce que la documentation respecte les lignes directrices de style d’écriture de Écrire la documentation?

  • Y a-t-il des fautes d’orthographe?

Bogues

  • Y a-t-il un test de régression approprié (le test doit échouer avant l’application de la correction) ?

  • S’il s’agit d’un bogue qui est susceptible d’être rétroporté vers la version stable de Django, existe-t-il une note de publication dans docs/releases/A.B.C.txt? Les corrections de bogues qui ne s’appliquent qu’à la branche main n’ont pas besoin d’une note de publication.

Nouvelles fonctionnalités

  • Y a-t-il des tests qui « éprouvent » la totalité du nouveau code ?

  • Existe-t-il une note de publication dans docs/releases/A.B.txt?

  • La fonctionnalité est-elle documentée et est-elle annotée convenablement avec .. versionadded:: A.B ou .. versionchanged:: A.B?

Obsolescence d’une fonctionnalité

Consultez le guide Obsolescence d’une fonctionnalité.

Toutes les modifications de code

  • Le style de code correspond-il à nos standards ? Y a-t-il des erreurs black, blacken-docs, flake8 ou isort? Vous pouvez installer les crochets ref:pre-commit <coding-style-pre-commit> pour intercepter automatiquement ces erreurs.

  • Si la modification n’est pas rétrocompatible d’une quelconque manière, existe-t-il une note de publication appropriée (dans docs/releases/A.B.txt) ?

  • Est-ce que la suite de tests de Django passe ?

Tous les tickets

Back to Top