Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Feature: Gerar um numero de celular aleatório #191 #232

Merged
merged 7 commits into from
Oct 10, 2023
Merged

Feature: Gerar um numero de celular aleatório #191 #232

merged 7 commits into from
Oct 10, 2023

Conversation

jhonatacaiob
Copy link
Contributor

@jhonatacaiob jhonatacaiob commented Oct 7, 2023 •
edited
Loading

Descrição

Cria um método que gera um numero de telefone aleatório e válido

>>> generate_mobile_phone()
'92942632303'
>>> generate_mobile_phone()
'17989047396'
>>> generate_mobile_phone()
'49997697725'
>>> generate_mobile_phone()
'85933479797'
>>> generate_mobile_phone()
'26979400345'

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

#191

Closes #191

@jhonatacaiob jhonatacaiob requested review from a team as code owners October 7, 2023 12:36
@codecov
Copy link

codecov bot commented Oct 7, 2023 •
edited
Loading

Codecov Report

Merging #232 (8710ef5) into main (0fed838) will not change coverage.
The diff coverage is 100.00%.

@@            Coverage Diff            @@
##              main      #232   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            9         9           
  Lines          168       174    +6     
=========================================
+ Hits           168       174    +6
Files Coverage Δ
brutils/__init__.py 100.00% <ø> (ø)
brutils/phone.py 100.00% <100.00%> (ø)

@jhonatacaiob jhonatacaiob mentioned this pull request Oct 7, 2023
Closed
@camilamaia camilamaia changed the title Feature: Gerar um numero de telefone aleatório #191 Feature: Gerar um numero de celular aleatório #191 Oct 9, 2023
camilamaia
camilamaia requested changes Oct 9, 2023
View reviewed changes
Copy link
Member

@camilamaia camilamaia left a comment

Choose a reason for hiding this comment

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

De código tá 💯 ! Valeu pela contribuição.

Só acabou faltando atualizar os arquivos de documentação:

  • CHANGELOG.md
  • README.md
  • README_EN.md

@jhonatacaiob
Copy link
Contributor Author

De código tá 💯 ! Valeu pela contribuição.

Só acabou faltando atualizar os arquivos de documentação:

  • CHANGELOG.md
  • README.md
  • README_EN.md

Obrigado pelas sugestões @camilamaia

Apliquei as sugestões e atualizei o PR

camilamaia
camilamaia reviewed Oct 10, 2023
View reviewed changes
Copy link
Member

@camilamaia camilamaia left a comment

Choose a reason for hiding this comment

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

Arrumando erros que inseri com o merge

brutils/__init__.py Outdated Show resolved Hide resolved
brutils/phone.py Outdated Show resolved Hide resolved
camilamaia
camilamaia reviewed Oct 10, 2023
View reviewed changes
brutils/__init__.py Outdated Show resolved Hide resolved
@camilamaia camilamaia merged commit 96ae0c9 into brazilian-utils:main Oct 10, 2023
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@camilamaia camilamaia camilamaia approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Gerar Número de Telefone Celular Válido e Aleatório
2 participants
@jhonatacaiob @camilamaia