email.py update - #249
Conversation
|
💩 Code linting failed, use |
|
посмотри таск который прикрепил к этому пр плюс надо под новую логику поправить тесты pytest и линтинг (форматирование кода с black/isort исправить, делается командой make format) |
petrCher
left a comment
petrCher
left a comment
There was a problem hiding this comment.
еще надо бы после всех исправлений потестить локально что все работает, то есть прям пройтись по бизнес логике ручек и посмотреть, что работает проверка email корректно, но делать это только после поправления тестов
как работать с тестами описано тут
| EMAIL_CONST_REGEX = re.compile( | ||
| r'^[a-zA-Z0-9_.+\\-%]+@[a-zA-Z0-9\\-]+\\.[a-zA-Z0-9\\-.]+$') | ||
|
|
||
| def email_check(v): |
There was a problem hiding this comment.
а почему ты рандомно поменял название функции? так же код весь дальнейший сломается
было же check_email а стало email_check
очень вероятно тесты из-за этого и падают
|
и еще верхний коммент бы поправить под текущий пр, а то там три уже несуществующих таска висят) |
2 participants
Изменения
добавлена регулярка
создан email_check(v)
value error неверный формат email
исправлены
Check-List
blackиisortдля Back-End илиPrettierдля Front-End?