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.

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

Criando métodos de busca de CEP/endereço consumindo a API da ViaCEP #358

Closed
wants to merge 10 commits into from

Conversation

gtkacz
Copy link

@gtkacz gtkacz commented Jun 6, 2024

Descrição

Criação de novos métodos de busca de endereço/CEP com dados reais. Por exemplo, buscar o número de um CEP e pegar dados de cidade, logradouro, bairro, etc.

Mudanças Propostas

  • Utilitário get_address_from_cep, busca informações detalhadas de endereço a partir de um CEP.
  • Utilitário get_cep_information_from_address busca CEP e informações detalhadas de endereço a partir de um UF, cidade e rua.

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 #353

Copy link

codecov bot commented Jun 6, 2024

Codecov Report

Attention: Patch coverage is 99.08257% with 1 line in your changes missing coverage. Please review.

Project coverage is 99.77%. Comparing base (80e194f) to head (450ecc7).

Files Patch % Lines
brutils/cep.py 97.95% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##              main     #358      +/-   ##
===========================================
- Coverage   100.00%   99.77%   -0.23%     
===========================================
  Files           10       17       +7     
  Lines          332      436     +104     
===========================================
+ Hits           332      435     +103     
- Misses           0        1       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@gtkacz gtkacz marked this pull request as ready for review June 6, 2024 16:38
@gtkacz gtkacz requested review from a team as code owners June 6, 2024 16:38
@gtkacz
Copy link
Author

gtkacz commented Jun 6, 2024

@camilamaia @antoniamaia criei esse PR pq o PR do criador original da issue não estava de acordo com as especificações do projeto, e utilizava bibliotecas terceiras que não eram necessárias.

Copy link
Contributor

@antoniamaia antoniamaia left a comment

Choose a reason for hiding this comment

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

Ainda estou revisando seu PR, mas aqui tem alguns pontos que acho interessante dar uma olhada! Obrigada pela contribuição :)

CHANGELOG.md Outdated Show resolved Hide resolved
Comment on lines +45 to +57
- [is\_valid\_cpf](#is_valid_cpf)
- [format\_cpf](#format_cpf)
- [remove\_symbols\_cpf](#remove_symbols_cpf)
- [generate\_cpf](#generate_cpf)
- [CNPJ](#cnpj)
- [is_valid_cnpj](#is_valid_cnpj)
- [format_cnpj](#format_cnpj)
- [remove_symbols_cnpj](#remove_symbols_cnpj)
- [generate_cnpj](#generate_cnpj)
- [is\_valid\_cnpj](#is_valid_cnpj)
- [format\_cnpj](#format_cnpj)
- [remove\_symbols\_cnpj](#remove_symbols_cnpj)
- [generate\_cnpj](#generate_cnpj)
- [CEP](#cep)
- [is_valid_cep](#is_valid_cep)
- [format_cep](#format_cep)
- [remove_symbols_cep](#remove_symbols_cep)
- [generate_cep](#generate_cep)
- [is\_valid\_cep](#is_valid_cep)
- [format\_cep](#format_cep)
- [remove\_symbols\_cep](#remove_symbols_cep)
Copy link
Contributor

Choose a reason for hiding this comment

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

Todas as alterações de formatação neste arquivo foram propositais? ou foram feitas automaticamente? Se não forem, consegue adicionar apenas a adição do seu utilitário? :)

Copy link
Author

@gtkacz gtkacz Jun 21, 2024

Choose a reason for hiding this comment

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

Foi proposital sim, meu markdown preview tava mostrando as palavras entre _ como itálico, então eu usei os escape characters: https://stackoverflow.com/questions/35827838/how-to-show-underscores-symbol-in-markdown, mas se quiser posso remover tbm

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah, tranquilo! Pode deixar, sem problemas

README.md Show resolved Hide resolved
is_valid_phone(phone_number, type)

```

Argumentos:
Copy link
Contributor

Choose a reason for hiding this comment

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

Aqui não consegui entender :( será que foi erro de merge?

Copy link
Author

Choose a reason for hiding this comment

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

Isso foi coisa do meu autoformat mesmo, mas ele tenta seguir algum padrão: https://github.com/DavidAnson/markdownlint/blob/v0.34.0/doc/md031.md

Tiro ou nem?

Copy link
Contributor

Choose a reason for hiding this comment

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

beleza, pode deixar também!! :)

README_EN.md Show resolved Hide resolved
brutils/cep.py Outdated Show resolved Hide resolved
@gtkacz gtkacz closed this by deleting the head repository Jul 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Obter endereço a partir de um cep
2 participants