Feature: Formatando numero de telefone #189 by jhonatacaiob · Pull Request #231 · brazilian-utils/python

@jhonatacaiob

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

#189

Closes #189

@codecov

Codecov Report

Merging #231 (3e05cf6) into main (09c07b7) will not change coverage.
Report is 5 commits behind head on main.
The diff coverage is 100.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%> (ø)

... and 3 files with indirect coverage changes

@jhonatacaiob

camilamaia

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.

@jhonatacaiob

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

@jhonatacaiob

camilamaia

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.