We’re always grateful for contributions to Django’s code. Indeed, bug reports\nwith associated contributions will get fixed far more quickly than those\nwithout a solution.
\nSe você está corrigindo um problema realmente trivial, por exemplo, mudando uma palavra na documentação, a melhor maneira de providenciar o patch é usando um pull request sem um ticket no Trac associado.
\nVeja a Trabalhando com Git e GitHub para mais detalhes sobre como usar os pull requests.
\nEm um projeto open-source com voluntários espalhados por todo o mundo, é importante gerenciar a comunicação de forma eficiente de modo que o trabalho não seja duplicado e voluntários possam ser o mais eficiente quanto possível.
\nPor tanto, nossa política é que os voluntários “reinvindiquem” os tickets de modo a permitir que os desenvolvedores do projeto saibam que aquele bug em específico ou funcionalidade está sendo atendida.
\nSe você identificou uma contribuição que você gostaria de fazer e se você é capaz de fazer essa correção (capacidade aqui mensurada pelas suas abilidades para programar, conhecimento de como o Django funciona internamente e tempo disponível), reinvindique ele seguindo os passos abaixo:
\nFaça login usando a sua conta no GitHub ou criando uma conta no nosso sistema de tickets. Se você tem uma conta mas esqueceu sua senha, você pode resetá-la usando a página de recuperação de senha.
Se um ticket para um problema não foi criado ainda, crie um no nosso rastreador de tickets.
Se o ticket para esse problema já existe, certifique-se de que ninguém mais o reinvindicou. Para fazer isso, procure na seção “Owned by” do ticket. Se ele estiver atribuído para “nobody”, então ele está disponível para reinvindicação. Caso contrário, outra pessoa já pode estar trabalhando nesse ticket. Busque outro bug/funcionalidade para trabalhar em cima ou contate o desenvolvedor trabalhando nesse ticket para oferecer a sua ajuda. Se o ticket já foi atribuído por semanas ou meses sem qualquer atividade, provavelmente é seguro reatribuí-lo para você.
Log into your account, if you haven’t already, by clicking “GitHub Login”\nor “DjangoProject Login” in the upper left of the ticket page. Once logged\nin, you can then click the “Modify Ticket” button near the bottom of the\npage.
Claim the ticket by clicking the “assign to” radio button in the “Action”\nsection. Your username will be filled in the text box by default.
Finally click the “Submit changes” button at the bottom to save.
Quando você reinvindicar um ticket, você terá a responsabilidade de trabalhar nele em uma quantidade de tempo razoável. Se você não tem tempo para trabalhar nele, ou abra mão dele ou nem se quer faça a reinvindicação do ticket em primeiro lugar!
\nSe não existir sinal de progresso em um ticket reinvindicado em particular por uma semana ou duas, outros desenvolvedores podem pedir para você abrir mão da reinvindicação do ticket de modo que ele não esteja mais monopolizado e alguém mais possa reinvindicá-lo novamente.
\nSe você reinvindicou um ticket e está demorando um bom tempo (dias ou semanas) para programar, mantenha todo mundo atualizado postando comentários no ticket. Se você não fornecer atualizações regulares, e você não responder a um pedido por relatório de progresso, a sua reinvindicação ao ticket poderá ser revogada.
\nComo sempre, mais comunicação é melhor do que menos comunicação.
\nGoing through the steps of claiming tickets is overkill in some cases.
\nIn the case of small changes, such as typos in the documentation or small bugs\nthat will only take a few minutes to fix, you don’t need to jump through the\nhoops of claiming tickets. Submit your changes directly and you’re done!
\nIt is always acceptable, regardless whether someone has claimed it or not, to\nlink proposals to a ticket if you happen to have the changes ready.
\nCertifique-se de que qualquer contribuição que você fizer preencha pelo menos os requisitos abaixo:
\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 validate the behavior that has been\nfixed and to prevent the problem from arising again. Also, if some tickets\nare relevant to the code that you’ve written, mention the ticket numbers in\nsome comments in the test so that one can easily trace back the relevant\ndiscussions after your patch gets committed, and the tickets get closed.
If the code adds a new feature, or modifies the behavior of an existing\nfeature, the change should also contain documentation.
When you think your work is ready to be reviewed, send a GitHub pull\nrequest.\nIf you can’t send a pull request for some reason, you can also use patches in\nTrac. When using this style, follow these guidelines.
\nEnvie os patches dentro do formato retornado por um comando git diff.
keAnexe os patches a um ticket dentro do ticket tracker, usando o botão “attach file”. Por favor não coloque o patch na descrição do ticket ou dentro de um comentário a não ser que seja um patch de uma linha.
Nomeie o arquivo de patch com a extensão .diff; isso permitirá que o rastreador de tickets aplique o estilo de formatação de sintaxe correto, o que ajuda muito.
Independentemente da maneira pela qual você irá enviar o seu trabalho, siga os passos a seguir:
\nMake sure your code fulfills the requirements in our contribution\nchecklist.
Marque a caixa “Has patch” no ticket e certifique-se de que as caixas “Needs documentation”, “Needs tests”, and “Patch needs improvement” não estão marcadas. Isso fará com que o ticket seja exibido na pilha “Patches needing review” na Development dashboard.
A wider community discussion is required when a patch introduces new Django\nfunctionality and makes some sort of design decision. This is especially\nimportant if the approach involves a deprecation\nor introduces breaking changes.
\nThe following are different approaches for gaining feedback from the community.
\nYou can propose a change on the Django Forum. You should explain the need\nfor the change, go into details of the approach and discuss alternatives.
\nPlease include a link to such discussions in your contributions.
\nDjango does not accept experimental features. All features must follow our\ndeprecation policy. Hence, it can\ntake months or years for Django to iterate on an API design.
\nIf you need user feedback on a public interface, it is better to create a\nthird-party package first. You can iterate on the public API much faster, while\nalso validating the need for the feature.
\nOnce this package becomes stable and there are clear benefits of incorporating\naspects into Django core, starting a discussion on the Django Forum would be\nthe next step.
\nSimilar to Python’s PEPs, Django has Django Enhancement Proposals or DEPs. A\nDEP is a design document which provides information to the Django community, or\ndescribes a new feature or process for Django. They provide concise technical\nspecifications of features, along with rationales. DEPs are also the primary\nmechanism for proposing and collecting community input on major new features.
\nBefore considering writing a DEP, it is recommended to first open a discussion\non the Django Forum. This allows the community to provide feedback and helps\nrefine the proposal. Once the DEP is ready the Steering Council votes on whether to accept it.
\nSome examples of DEPs that have been approved and fully implemented:
\n\nThere are a couple of reasons that code in Django might be deprecated:
\nSe a funcionalidade foi melhorada ou modificada de modo a não ser mais compatível com versões anteriores, a antiga funcionalidade ou comportamento pode ser depreciada.
Algumas vezes o Django inclui um backport de uma biblioteca Python que não está incluída na versão do Python suportada atualmente pelo Django. Quando o Django não precisa mais dar suporte a versão antiga do Python que não fornece a biblioteca, essa biblioteca será depreciada no Django.
As the deprecation policy describes,\nthe first release of Django that deprecates a feature (A.B) should raise a\nRemovedInDjangoXXWarning (where XX is the Django version where the feature\nwill be removed) when the deprecated feature is invoked. Assuming we have good\ntest coverage, these warnings are converted to errors when running the\ntest suite with warnings enabled:\npython -Wa runtests.py. Thus, when adding a RemovedInDjangoXXWarning\nyou need to eliminate or silence any warnings generated when running the tests.
O primeiro passo é remover qualquer uso do comportamento depreciado no próprio Django. Depois você pode silenciar os warnings nos testes que testam o comportamento depreciado usando o decorator ignore_warnings, na classe ou diretamente nos testes:
Em um teste em particular:
\nfrom django.test import ignore_warnings\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\n\n@ignore_warnings(category=RemovedInDjangoXXWarning)\ndef test_foo(self): ...\nPara um caso de teste inteiro:
\nfrom django.test import ignore_warnings\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\n\n@ignore_warnings(category=RemovedInDjangoXXWarning)\nclass MyDeprecatedTests(unittest.TestCase): ...\nYou should also add a test for the deprecation warning:
\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__)\nIt’s important to include a RemovedInDjangoXXWarning comment above code\nwhich has no warning reference, but will need to be changed or removed when the\ndeprecation ends. This could include hooks which have been added to keep the\nprevious behavior, or standalone items that are unnecessary or unused when the\ndeprecation ends. For example:
import warnings\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\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 stacklevel=2,\n )\n old_private_helper()\n ...\nFinalmente, existem algumas atualizações na documentação do Django para fazer:
\nSe a funcionalidade existente está documentada, marque ela como depreciada na documentação usando a anotação .. deprecated:: A.B. Inclua uma curta descrição e uma nota sobre o caminho para atualizar se possível.
Adicione uma descrição de comportamento depreciado, e o caminho de atualização se possível, para as notas da release atual (docs/releases/A.B.txt) abaixo do header “Features deprecated in A.B”.
Adicione uma nova entrada na linha do tempo de depreciação (docs/internals/deprecation.txt) abaixo da versão apropriada descrevendo qual código será removido.
Once you have completed these steps, you are finished with the deprecation.\nIn each feature release, all\nRemovedInDjangoXXWarnings matching the new version are removed.
For information on JavaScript contributions, see the Patches JavaScript\ndocumentation.
\nPatches aiming to deliver a performance improvement should provide benchmarks\nshowing the before and after impact of the patch and sharing the commands for\nreviewers to reproduce.
\ndjango-asv benchmarksLink para este cabeçalho#django-asv monitors the performance of Django code over time. These\nbenchmarks can be run on a pull request by labeling the pull request with\nbenchmark. Adding to these benchmarks is highly encouraged.
Use this checklist to review a pull request. If you are reviewing a pull\nrequest that is not your own and it passes all the criteria below, please set\nthe “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 permits, 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. If you’re looking to become a merger, doing thorough\nreviews of contributions is a great way to earn trust.
\nLooking for a patch to review? Check out the “Patches needing review” section\nof the Django Development Dashboard.
\nLooking to get your pull request reviewed? Ensure the Trac flags on the ticket\nare set so that the ticket appears in that queue.
\nA documentação é gerada sem erros (make html, ou make.bat html no Windows, do diretório docs)?
A documentação segue as regras de estilo e escrita em Escrevendo a documentação?
Existem quaisquer erros de ortografia?
Existe um teste de regressão apropriado (o teste deve falhar antes da correção ser aplicada)?
If it’s a bug that qualifies for a backport\nto the stable version of Django, is there a release note in\ndocs/releases/A.B.C.txt? Bug fixes that will be applied only to the main\nbranch don’t need a release note.
Existem testes para “exercitar” todo o código recém-criado?
Existe uma nota de release em docs/releases/A.B.txt?
Existe documentação para a funcionalidade e ela está anotada de modo apropriado com .. versionadded:: A.B ou .. versionchanged:: A.B?
Veja o guia depreciando uma funcionalidade.
\nDoes the coding style conform to our\nguidelines? Are there any black, blacken-docs, flake8,\nisort, or zizmor errors? You can install the pre-commit hooks to automatically catch these errors.
Se a mudança não é compatível com versões anteriores de modo algum, existe um aviso nas notas de release (docs/releases/A.B.txt)?
Is Django’s test suite passing?
O pull request é um commit único esmagado com uma mensagem que segue o nosso formato de mensagens de commit?
Are you the patch author and a new contributor? Please add yourself to the\nAUTHORS file and submit a Contributor License Agreement.