-
-
Notifications
You must be signed in to change notification settings - Fork 109
docs: ajusta formatação #675
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
base: main
Are you sure you want to change the base?
Conversation
| - [CEP](#cep) | ||
| - [is\_valid\_cep](#is_valid_cep) | ||
| - [format\_cep](#format_cep) | ||
| - [remove\_symbols\_cep](#remove_symbols_cep) |
There was a problem hiding this comment.
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.... 🤔
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
| - 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) |
There was a problem hiding this comment.
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 🙏 🙇♀️
There was a problem hiding this comment.
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
There was a problem hiding this comment.
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
Descrição
Ajustada formatação do arquivo
CHANGELOG.mde links corretos com as respectivas issues.Mudanças Propostas
Checklist de Revisão
Comentários Adicionais (opcional)
Issue Relacionada
Closes #674