feat: add is_email_valid by Anurag-Nagpal · Pull Request #213 · brazilian-utils/python
Navigation Menu
{{ message }}
brazilian-utils / python Public
- Notifications You must be signed in to change notification settings
- Fork 113
Merged
antoniamaia merged 4 commits intobrazilian-utils:mainfrom
Oct 4, 2023Merged
antoniamaia merged 4 commits intobrazilian-utils:mainfrom
antoniamaia merged 4 commits intobrazilian-utils:mainfrom
Conversation
Copy link
Contributor
Descrição
Mudanças Propostas
Checklist de Revisão
- [yes] Eu li o Contributing.md
- [yes] Os testes foram adicionados ou atualizados para refletir as mudanças (se aplicável).
- [yes] Foi adicionada uma entrada no changelog / Meu PR não necessita de uma nova entrada no changelog.
- [yes] A documentação em português foi atualizada ou criada, se necessário.
- [yes] Se feita a documentação, a atualização do arquivo em inglês.
- [yes] Eu documentei as minhas mudanças no código, adicionando docstrings e comentários. Instruções
- [yes] O código segue as diretrizes de estilo e padrões de codificação do projeto.
- [yes] Todos os testes passam. Instruções
- [yes] O Pull Request foi testado localmente. Instruções
- [yes] Não há conflitos de mesclagem.
Comentários Adicionais (opcional)
Issue Relacionada
Closes #179
Anurag-Nagpal
requested review from
a team
as code owners
Copy link
Codecov Report
Merging #213 (c267ef6) into main (c0044a5) will not change coverage.
The diff coverage is100.00%.
@@ Coverage Diff @@ ## main #213 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 5 6 +1 Lines 93 98 +5 ========================================= + Hits 93 98 +5
| Files | Coverage Δ | |
|---|---|---|
| brutils/__init__.py | 100.00% <100.00%> (ø) |
|
| brutils/email.py | 100.00% <100.00%> (ø) |
Anurag-Nagpal and others added 3 commits
October 4, 2023 20:52
Anurag-Nagpal
mentioned this pull request
Closed
antoniamaia approved these changes Oct 4, 2023
Copy link
Member
antoniamaia
left a comment
antoniamaia
left a comment
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Well done, @Anurag-Nagpal!! Thanks for your contribution 🥳
antoniamaia
merged commit
c69e6fb
into
brazilian-utils:main
antoniamaia
mentioned this pull request
Merged
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment