Nós estamos sempre gratos por patches para o código do Django. De fato, relatos de bugs com um respectivo patch serão corrigidos muito mais rápido do que os bugs relatados sem um patch.
\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ê.
Acesse a sua conta, se você não tem uma ainda, clicando em “GitHub Login” ou “DjangoProject Login” no topo esquerdo da página de tickets.
Reinvindique o ticket clicando na opção “assign to myself” em “Action” quase no final da página do ticket, e depois clique em “Submit changes”.
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 patch directly and you’re done!
\nIt is always acceptable, regardless whether someone has claimed it or not, to\nsubmit patches to a ticket if you happen to have a patch ready.
\nCertifique-se de que qualquer contribuição que você fizer preencha pelo menos os requisitos abaixo:
\nO código necessário para corrigir um problema ou adicionar uma funcionalidade é uma parte essencial de um patch, mas ele não é a única. Um bom patch também deve incluir um teste de regressão para validar o comportamento que foi corrigido e para prevenir o problema de aparecer novamente. Além disso, se alguns tickets são relevantes para o código que você escreveu, mencione os números desses tickets em algum comentário do teste de modo que alguém possa facilmente encontrar as discussões relevantes depois do seu patch ser commitado, e o ticket for fechado.
Se o código associado com o patch adiciona uma nova funcionalidade, ou modifica o comportamento de uma funcionalidade existente, o patch também deve conter documentação.
Quando você achar que o seu trabalho está pronto para ser revisado, envie um pull request pelo GitHub. Por favor revise o ptch você mesmo usando nossa checklist de revisão de patchs primeiro.
\nSe você não pode enviar um pull request por alguma razão, você também pode usar patches dentro do Trac. Quando usar esse estilo, siga as orientações a seguir:
\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:
\nCertifique-se de que o seu código preenche todos os requisitos em nosso checklist de revisão de patchs.
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 “non-trivial” patch is one that is more than a small bug fix. It’s a patch\nthat introduces Django functionality and makes some sort of design decision.
\nSe você estiver enviando um patch não trivial, inclua evidências de que alternativas foram discutidas na lista django-developers.
\nIf you’re not sure whether your patch should be considered non-trivial, ask on\nthe ticket for opinions.
\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@ignore_warnings(category=RemovedInDjangoXXWarning)\ndef test_foo(self):\n ...\nPara um caso de teste inteiro:
\nfrom django.test import ignore_warnings\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\n@ignore_warnings(category=RemovedInDjangoXXWarning)\nclass MyDeprecatedTests(unittest.TestCase):\n ...\nYou can also add a test for the deprecation warning:
\nfrom django.utils.deprecation import RemovedInDjangoXXWarning\n\ndef test_foo_deprecation_warning(self):\n msg = 'Expected deprecation message'\n with self.assertWarnsMessage(RemovedInDjangoXXWarning, msg):\n # invoke deprecated behavior\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.
para informação sobre patches em JavaScript, veja a documentação Patches JavaScript .
\nUse 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, committers do final reviews of “Ready for checkin”\ntickets and will either commit the patch or bump it back to “Accepted” if\nfurther works need to be done. If you’re looking to become a committer,\ndoing thorough reviews of patches is a great way to earn trust.
\nProcurando um patch para revisar? Veja a seção “Patches needing review” do Django Development Dashboard. Querendo que o patch que você desenvolveu seja revisado? Certifique-se de que as flags do ticket no Trac foram setadas corretamente de modo que ele apareça nessa fila.
\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\nmaster branch 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.
\nO estilo de código atende as nossas regras? Existem quaisquer erros gerados pelo flake8?
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?
Você é o autor e um novo voluntário? Por favor se adicione no arquivo AUTHORS e envie um Contributor License Agreement.