Skip to content

Conversation

@lucasjl
Copy link

@lucasjl lucasjl commented Oct 27, 2025

Descrição

Este PR resolve a issue #620 , migrando type hints do arquivo license_plate.py de acordo com PEP 484, utilizando anotações modernas

Mudanças Propostas

  • Substituir Optional[str] por 'str | None' em anotações de funções.
  • Ajustar docstrings que mencionam Optional, substituindo pelas anotações modernas.
  • Uso do Literal para especificar 'old_format' ou 'mercosul' no argumento 'type' da função 'is_valid'

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)

N/A

Issue Relacionada

Closes #620

@lucasjl lucasjl requested review from a team as code owners October 27, 2025 17:43
@lucasjl lucasjl changed the title Migrar type hints de license_plate.py de acordo com PEP 484 refactor: Migrar type hints de license_plate.py de acordo com PEP 484 Oct 27, 2025
Copy link
Contributor

@morais90 morais90 left a comment

Choose a reason for hiding this comment

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

Boa @lucasjl ! Só faltou atualizar o CHANGELOG.md

@lucasjl
Copy link
Author

lucasjl commented Oct 30, 2025

Boa @lucasjl ! Só faltou atualizar o CHANGELOG.md

Muito obrigado pela revisão, Willian!

Sobre o Changelog, ali durante o dia da sprint comentaram que esses mudanças nos type hints seriam classificadas como mudanças internas do código e não precisariam ser adicionadas ao CHANGELOG. Vendo o escopo da issue e as mudanças, realmente não vejo nada público, aí acabei não adicionando mesmo.

O que tu acha? Posso adicionar sem problemas, só queria confirmar contigo.

Abraço!

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.

Migrar type hints de brutils/license_plate.py para anotações modernas (PEP 484)

2 participants