Feature: Formatando numero de telefone #189 by jhonatacaiob · Pull Request #231 · brazilian-utils/python
Descrição
Criando método para formatar um numero de telefone válido para um formato legível
>>> format_phone("11994029275") '(11)99402-9275' >>> format_phone("1635014415") '(16)3501-4415' >>> format_phone("333333") >>>
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.
Issue Relacionada
Closes #189
Codecov Report
Merging #231 (3e05cf6) into main (09c07b7) will not change coverage.
Report is 5 commits behind head on main.
The diff coverage is100.00%.
❗ Current head 3e05cf6 differs from pull request most recent head 439259d. Consider uploading reports for the commit 439259d to get more accurate results
@@ Coverage Diff @@ ## main #231 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 9 9 Lines 149 161 +12 ========================================= + Hits 149 161 +12
| Files | Coverage Δ | |
|---|---|---|
| brutils/__init__.py | 100.00% <ø> (ø) |
|
| brutils/phone.py | 100.00% <100.00%> (ø) |
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
O código tá 💯, valeu pela contribuição!
Só deixei duas pequenas sugestões na parte de documentação. Precisaria também adicionar as atualizações dos arquivos README.md e README_EN.md com o novo utilitário format_phone.
O código tá 💯, valeu pela contribuição!
Só deixei duas pequenas sugestões na parte de documentação. Precisaria também adicionar as atualizações dos arquivos README.md e README_EN.md com o novo utilitário format_phone.
Obrigado pelas sugestões @camilamaia
Apliquei as sugestões e atualizei o PR
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters