World's NIF

[carolcardoso080]

Descrição

Mudanças Propostas

Checklist de Revisão

• [ ] Eu li o Contributing.md
• [ ] Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
• [ ] Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
• [ ] A documentação em português foi atualizada ou criada, se necessário.
• [ ] Se feita a documentação, a atualização do arquivo em inglês.
• [ ] Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
• [ ] O código segue as diretrizes de estilo e padrões de codificação do projeto.
• [ ] Todos os testes passam. Instruções
• [ ] O Pull Request foi testado localmente. Instruções
• [ ] Não há conflitos de mesclagem.

Comentários Adicionais (opcional)

Issue Relacionada

Closes #<numero_da_issue>

[codecov[bot]]

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 99.80%. Comparing base (d7b645d) to head (385803a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #457   +/-   ##
=======================================
  Coverage   99.80%   99.80%           
=======================================
  Files          19       19           
  Lines         521      521           
=======================================
  Hits          520      520           
  Misses          1        1           

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

[carolcardoso080]

@camilamaia

Desculpa, a intenção não era já pedir para dar merge agora. Estava seguindo o passo a passo do Contributing e acho que deu nisso.

Depois, se você conseguir dar uma olhada nos 2 códigos (de 2 países distintos), você me dá um retorno se está correto ou quais modificações seguir para manter o padrão!

Assim eu posso dar continuidade nos próximos da mesma forma.

[camilamaia]

Oii @carolcardoso080!

Imagina, sem problemas! Se este pull request ainda não está pronto para ser mesclado e você deseja que ele seja apenas revisado ou analisado, é uma boa prática marcá-lo como rascunho (draft). Isso ajuda a sinalizar que o trabalho está em andamento e que o pull request ainda não está pronto para ser integrado.

Como converter um pull request existente para rascunho

• Vá até o seu pull request.
• Na barra lateral direita, na seção "Reviewers", clique em "Convert to draft".

Para instruções detalhadas, veja a documentação oficial do GitHub.

Como marcar um draft como pronto para revisão

Depois, quando você tiver já o seu PR pronto para realmente ser revisado para um merge, faça o seguinte:

• Abra o seu PR
• Desça até a caixa de mesclagem e clique em "Ready for Review".

Para instruções detalhadas, veja a documentação oficial do GitHub.

Como criar um pull request como rascunho

Caso já queira criar um PR como draft, no momento de criar o pull request, clique na seta ao lado do botão "Create pull request" e selecione "Create draft pull request".

Para mais detalhes, consulte a documentação oficial do GitHub.

Se precisar de ajuda nessa parte, é só chamar! 😊 Agora vou dar uma olhadinha no código!

[camilamaia]

@carolcardoso080 ah, percebi mais um detalhezinho agora vendo de novo. Os arquivos terminados em .py sempre vão utilizar o underscore _ para separação de palavras no projeto. Os arquivos vão ficar assim então:

• brutils/worlds_nif/nif_haiti.py
• brutils/worlds_nif/nif_portugal.py

[camilamaia]

Oi, @carolcardoso080 ! Estou fechando o PR por inatividade, mas fique à vontade para reabrir caso queira retomar o trabalho. 😊