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.
\nSi 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.
\nSee Travailler avec Git et GitHub for more\ndetails on how to use pull requests.
\nDans 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.
\nAinsi 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.
\nSi 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 :
\nLog in using your GitHub account or create an account in our ticket\nsystem. If you have an account but have forgotten your password, you can\nreset it using the password reset page.
If a ticket for this issue doesn’t exist yet, create one in our\nticket tracker. Remember that proposals for new features should follow\nthe process for suggesting new features.
If a ticket for this issue already exists and has been accepted, make sure\nnobody else has claimed it. To do this, look at the « Owned by » section of\nthe ticket. If it’s assigned to « nobody, » then it’s available to be claimed.\nOtherwise, somebody else may be working on this ticket. Either find another\nbug/feature to work on, or contact the developer working on the ticket to\noffer your help. If a ticket has been assigned for weeks or months without\nany activity, it’s probably safe to reassign it to yourself. If a ticket\nhasn’t been approved yet, join the conversation.
Connectez-vous si ce n’est pas déjà fait, en cliquant sur « GitHub Login » ou « DjangoProject Login » dans la partie supérieure gauche de la page du ticket. Une fois connecté, vous pouvez cliquer sur le bouton « Modifier le ticket » au bas de la page.
Attribuez-vous le ticket en cliquant sur le bouton radio « assign to » dans la section « Action ». Votre nom d’utilisateur sera ajouté par défaut dans la zone de texte.
Finally, click the « Submit changes » button at the bottom to save.
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 !
\nS’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.
\nSi 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.
\nComme toujours, mieux vaut trop communiquer que pas assez !
\nPasser par l’étape d’attribution de tickets est parfois surfait.
\nDans 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à !
\nIt is always acceptable, regardless of whether someone has claimed it or not,\nto link proposals to a ticket if you happen to have the changes ready.
\nAssurez-vous que toute contribution remplit au minimum les exigences suivantes :
\nThe code required to fix a problem or add a feature is an essential part\nof a solution, but it is not the only part. A good fix should also include a\nregression test to\nvalidate the behavior that has been fixed and to prevent the problem from\narising again. Also, if some tickets are relevant to the code that you’ve\nwritten, mention the ticket numbers in some comments in the test so that one\ncan easily trace back the relevant discussions after your patch gets\ncommitted, and the tickets get closed.
Si le code ajoute une nouvelle fonctionnalité ou modifie le comportement d’une fonctionnalité existante, la modification doit aussi contenir de la documentation.
When you think your work is ready to be reviewed, send a GitHub pull\nrequest. If you can’t\nsend a pull request for some reason, you can also use patches in Trac. When\nusing this style, follow these guidelines.
\nEnvoyez 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.
\nAssurez-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.
Une discussion dans la communauté élargie est nécessaire lorsqu’un correctif introduit une nouvelle fonctionnalité Django et donne ainsi lieu à une forme de décision conceptuelle. C’est particulièrement important quand l’approche choisie implique une obsolescence ou introduit des modifications disruptives.
\nVoici différentes approches pour obtenir des retours de la communauté.
\nIf you have an idea for a new feature, please create a new proposal (or join an\nexisting discussion) following the process for suggesting new features. You should explain the need for the change, go into\ndetails of the approach and discuss alternatives.
\nYou can propose a change (that is not a new feature idea) on the\nDjango Forum. You should explain the need for the change, go into details of\nthe approach and discuss alternatives.
\nNous vous prions d’inclure un lien vers ces discussions dans vos contributions.
\nDjango n’accepte pas de fonctionnalités expérimentales. Toutes les fonctionnalités doivent suivre notre politique d’obsolescence. Cela signifie qu’il peut se passer des mois ou des années pour que Django modifie l’API d’une fonctionnalité.
\nSi vous attendez un retour d’utilisation sur une interface publique, il est préférable de créer d’abord un paquet tiers. Il est possible d’ajuster l’API publique de manière bien plus rapide tout en validant le besoin réel de la fonctionnalité.
\nOnce this package becomes stable and there are clear benefits of incorporating\naspects into Django core, the next step is to propose its inclusion by\nfollowing the process for suggesting new features.
\nSur le modèle des PEP de Python, Django possède un concept de propositions d’améliorations appelées DEP (Django Enhancement Proposals). Une DEP est un document conceptuel fournissant des informations à la communauté Django ou décrivant une fonctionnalité ou un processus nouveaux pour Django. Elles présentent de brèves spécifications techniques de fonctionnalités, accompagnées des motivations du besoin. Les DEP sont également le mécanisme principal pour proposer et collecter des avis de la communauté sur de nouvelles fonctionnalités majeures.
\nBefore considering writing a DEP, it is recommended to first open a discussion\nfollowing the process for suggesting new features.\nThis allows the community to provide feedback and helps refine the proposal.\nOnce the DEP is ready, the Steering Council votes on\nwhether to accept it.
\nVoici quelques exemples de DEP approuvées et complètement implémentées :
\n\nIl existe plusieurs raisons pour que du code Django soit rendu obsolète :
\nSi 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 :
Pour un test particulier
\nfrom django.test import ignore_warnings\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\n\n@ignore_warnings(category=RemovedInDjangoXXWarning)\ndef test_foo(self): ...\nPour tout un cas de test
\nfrom django.test import ignore_warnings\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\n\n@ignore_warnings(category=RemovedInDjangoXXWarning)\nclass MyDeprecatedTests(unittest.TestCase): ...\nVous devriez aussi ajouter un test pour l’avertissement d’obsolescence
\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\n\ndef test_foo_deprecation_warning(self):\n msg = "Expected deprecation message"\n with self.assertWarnsMessage(RemovedInDjangoXXWarning, msg) as ctx:\n # invoke deprecated behavior\n ...\n self.assertEqual(ctx.filename, __file__)\nIl 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\nfrom django.utils.deprecation import RemovedInDjangoXXWarning, django_file_prefixes\n\n\n# RemovedInDjangoXXWarning.\ndef old_private_helper():\n # Helper function that is only used in foo().\n pass\n\n\ndef foo():\n warnings.warn(\n "foo() is deprecated.",\n category=RemovedInDjangoXXWarning,\n skip_file_prefixes=django_file_prefixes(),\n )\n old_private_helper()\n ...\nPour terminer, il reste à appliquer quelques mises à jour de la documentation Django :
\nSi 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.
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 ».
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.
The django.utils.deprecation module provides some helpful deprecation\nutilities, such as a @deprecate_posargs decorator to assist with converting\npositional-or-keyword arguments to keyword-only. See the inline documentation\nin the module source.
It’s important to test local changes using a Django project. This allows\nensuring that the changes behave as expected in a real environment, especially\nfor user-facing features such as templates, forms, or the admin.
\nTo do this:
\nCreate a virtual environment and install the cloned copy of Django in\neditable mode.
Set up a Django project outside the source tree (you can use the first\npart of the tutorial for guidance).
With this setup, any changes made to the Django checkout will take effect\nimmediately in the test project, allowing manual testing of contributions\nagainst a new or existing app.
\nPour des informations sur les contributions JavaScript, consultez la documentation Correctifs JavaScript.
\nLes 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.
\ndjango-asvLien vers cette rubrique#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é.
Use this checklist to review a pull request. If this contribution would not be\nconsidered trivial, first ensure it has an accepted\nticket before proceeding with the review.
\nIf the pull request passes all the criteria below and is not your own, please\nset the « Triage Stage » on the corresponding Trac ticket to « Ready for checkin ».\nIf you’ve left comments for improvement on the pull request, please tick the\nappropriate flags on the Trac ticket based on the results of your review:\n« Patch needs improvement », « Needs documentation », and/or « Needs tests ». As time\nand interest permit, mergers do final reviews of « Ready for checkin » tickets\nand will either commit the changes or bump it back to « Accepted » if further\nwork needs to be done.
\nIf you’re looking to become a member of the triage & review team, doing\nthorough reviews of contributions is a great way to earn trust.
\nEn recherche d’un correctif à relire ? Consultez la section « Patches needing review » du tableau de bord de développement de Django.
\nVous 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.
\nLa requête de contribution est-elle un seul commit fusionné avec un message respectant notre format des messages de commit?
Are you the patch author and a new contributor? Please add yourself to the\nAUTHORS file. At your option, submit a\nContributor License Agreement.
Does this have an accepted ticket on Trac? All contributions require a ticket\nunless the change is considered trivial.
Le style de code correspond-il à nos standards ? Y a-t-il des erreurs black, blacken-docs, flake8, isort ou zizmor? 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 ?
If there is a code coverage report\ncomment on the pull request, have you reviewed the missing coverage in\ncontext (considering database/platform-specific limitations)?
If the change affects the Django admin or rendered HTML output, has\naccessibility testing been done?
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?
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.
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?
Consultez le guide Obsolescence d’une fonctionnalité.
\n