Skip to content

Conversation

@niltonpimentel02
Copy link
Member

Descrição

Ajustada formatação do arquivo CHANGELOG.md e links corretos com as respectivas issues.

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

@niltonpimentel02 niltonpimentel02 requested review from a team as code owners December 15, 2025 14:30
- [CEP](#cep)
- [is\_valid\_cep](#is_valid_cep)
- [format\_cep](#format_cep)
- [remove\_symbols\_cep](#remove_symbols_cep)
Copy link
Member

Choose a reason for hiding this comment

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

Talvez eu esteja viajando aqui, mas por que a remoção desse utilitário do índice? Vi que tirou também do readme em inglês.... 🤔

Copy link
Member Author

Choose a reason for hiding this comment

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

essa função foi removida e simplificada nesse último pr https://github.com/brazilian-utils/python/pull/668

Copy link
Member

@camilamaia camilamaia Dec 21, 2025

Choose a reason for hiding this comment

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

Para mim só adicionar essa linha de volta nos dois README's e dale!

contexto: #678

Comment on lines -21 to -29
- Suporte a anotações modernas no `brutils/cep.py` [#637](https://github.com/brazilian-utils/python/pull/637)

### Fixed

- Suporte a anotações modernas `legal_process` [#622](https://github.com/brazilian-utils/brutils-python/pull/634)

### Fixed

- Suporte a anotações modernas `voter_id` [#623](https://github.com/brazilian-utils/brutils-python/pull/638)
Copy link
Member

Choose a reason for hiding this comment

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

Boa demais, tava dando aflição mesmo os fixed repetidos haha 🙏 🙇‍♀️

Copy link
Member

Choose a reason for hiding this comment

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

O padrão que a gente tava seguindo nesse CHANGELOG era de colocar os links do PR. Eu pensei nisso porque daí vc consegue ver direta e exatamente a mudança que foi de fato implementada. E se quiser voltar para entender melhor o contexto, pode clicar na issue linkada no PR.

Mas para ti faz mas sentido linkar a issue? Se for o caso, podemos pensar em mudar também

Copy link
Member Author

Choose a reason for hiding this comment

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

ah eu pensei que era o contrário mas tranquilo eu mudo aqui.. to fazendo sem pressa pra deixar tudo de volta aos trilhos pra iniciar 2026 zeradinho haha

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.

Ajustar formatação CHANGELOG

2 participants